From <kaleb@redhat.com>: kaleb@redhat.com has uploaded this change for review. ( https://review.gerrithub.io/419427 Change subject: build: location of xattr.h ...................................................................... build: location of xattr.h starting with libattr-devel-2.4.48-1 in Fedora 28 <attr/xattr.h> has been removed from the package. On Fedora, RHEL/CentOS, and SUSE, the glibc-headers package has provided a near identical file, <sys/xattr.h> in all the releases that we care about. On Debian, libc6-dev provides <sys/xattr.h> all the way back to 8/Jessie and presumably all Ubuntu derivatives since then. Note that on Debian the sys headers are installed in /usr/include/$DEB_HOST_GNU_TYPE/sys/... Above and beyond that there is an .../include/os/xattr.h that should be used. Fixes CMakelists.txt (for FSAL_GLUSTER) to look for xattr.h in .../sys/ and related config-h.in.cmake change, fixes FSAL_GLUSTER to use "os/xattr.h", and fixes several erroneous uses of <os/xattr.h> that should be "os/xattr.h" Change-Id: Ic0f334ed768484c12a868d9abd5126c69b7f7023 Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com> --- M src/CMakeLists.txt M src/FSAL/FSAL_GLUSTER/gluster_internal.c M src/FSAL/FSAL_VFS/xattrs.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_xattrs.c M src/FSAL/Stackable_FSALs/FSAL_NULL/xattrs.c M src/Protocols/9P/9p_xattrcreate.c M src/Protocols/9P/9p_xattrwalk.c M src/include/config-h.in.cmake 8 files changed, 15 insertions(+), 15 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/27/419427/1 -- To view, visit https://review.gerrithub.io/419427 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ic0f334ed768484c12a868d9abd5126c69b7f7023 Gerrit-Change-Number: 419427 Gerrit-PatchSet: 1 Gerrit-Owner: kaleb@redhat.com