On 07/16/18 15:21, Niels de Vos wrote:
 On Mon, Jul 16, 2018 at 02:46:12PM +0200, LUCAS Patrice wrote:
> On 07/16/18 14:20, Niels de Vos wrote:
>> On Mon, Jul 16, 2018 at 11:21:35AM +0200, patrice.lucas(a)cea.fr wrote:
>>> I'm trying to update Ganesha CEA Continuous-Integration tests by setting
a
>>> new configuration over FEDORA 28.
>>>
>>>
>>> I'm facing a first cmake error because the "attr/xattr.h" file
seems not to
>>> exist anymore in Fedora 28. This file is claimed by the gluster FSAL.
>>>
>>>
>>> Does anyone already compile the Gluster FSAL over Fedora 28 ?
>>>
>>>
>>> Should we remove the use of "attr/xattr.h" by the gluster fsal ?
Which is
>>> the bast replacement for "attr/xattr.h" ?
>> You should be able to install libattr-devel and get the headers. It
>> should get pulled in when you install glusterfs-api-devel that is needed
>> for FSAL_GLUSTER.
>>
>> HTH,
>> Niels
> libattr-devel is installed but the /usr/include/attr/xattr.h is missing. I'm
> still looking for an explanation of the following situation :
>
>
> [root@vm0 ~]# dnf provides "/usr/include/attr/xattr.h"
> Last metadata expiration check: 0:59:09 ago on Mon 16 Jul 2018 11:39:58 AM
> UTC.
> libattr-devel-2.4.47-23.fc28.i686 : Files needed for building programs with
> libattr
> Repo        : fedora
> Matched from:
> Filename    : /usr/include/attr/xattr.h
>
> libattr-devel-2.4.47-23.fc28.x86_64 : Files needed for building programs
> with libattr
> Repo        : fedora
> Matched from:
> Filename    : /usr/include/attr/xattr.h
>
> [root@vm0 ~]#
> [root@vm0 ~]#
> [root@vm0 ~]# rpm -ql libattr-devel
> /usr/include/attr
> /usr/include/attr/attributes.h
> /usr/include/attr/error_context.h
> /usr/include/attr/libattr.h
> /usr/lib64/libattr.so
> /usr/lib64/pkgconfig/libattr.pc
> /usr/share/man/man3/attr_get.3.gz
> /usr/share/man/man3/attr_getf.3.gz
> /usr/share/man/man3/attr_list.3.gz
> /usr/share/man/man3/attr_listf.3.gz
> /usr/share/man/man3/attr_multi.3.gz
> /usr/share/man/man3/attr_multif.3.gz
> /usr/share/man/man3/attr_remove.3.gz
> /usr/share/man/man3/attr_removef.3.gz
> /usr/share/man/man3/attr_set.3.gz
> /usr/share/man/man3/attr_setf.3.gz
> [root@vm0 ~]# rpm -q libattr-devel
> libattr-devel-2.4.48-1.fc28.x86_64
> [root@vm0 ~]#
>
>
> The "/usr/include/attr/xattr.h" seems to disappear between
> libattr-devel-2.4.47-23.fc28.x86_64 and libattr-devel-2.4.48-1.fc28.x86_64 .
> Do you have any idea why this could happen ?
 Oh, indeed. I do not have that file either. We'll need to check if it is
 still part of upstream libattr, or if it moved to something else.
    # dnf install /usr/include/attr/xattr.h
 Suggests it really is only available in libattr-devel-2.4.47-23.fc28 and
 not newer versions.
 The change was introduced with this commit:
   
http://git.savannah.nongnu.org/cgit/attr.git/commit/libattr?id=7921157890...
 It seems <sys/xattr.h> can be used instead. We might need to update
 libgfapi headers for this? You can maybe just symlink the header to
 /usr/include/attr/xattr.h as a workaround.
 Niels 
I opened a bug on 
bugzilla.redhat.com : 
https://bugzilla.redhat.com/show_bug.cgi?id=1601482 .
I can add a symlink as workaround but we need a true fix for the ganesha 
Gluster FSAL if this change is confirmed (we will see the result of the 
open bugzilla). If the attr/xattr.h is really useless and deprecated, 
what anyone suggest to remove it from the gluster fsal dependencies ?
-- 
Patrice LUCAS
Ingenieur-Chercheur, CEA-DAM/DSSI/SISR/LA2S
tel : +33 (0)1 69 26 47 86
e-mail : patrice.lucas(a)cea.fr