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

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