[Gluster-devel] build failure

[Ravishankar N]

On 02/12/2014 03:18 PM, Emmanuel Dreyfus wrote:
> Anyone can explain me why this one fails?
> http://build.gluster.org/job/smoke/6441/
>
> build console is below. It builds at mine, of course...
>
> libtool: install: warning: relinking `template.la'
> chown: changing ownership of `/build/install/bin/fusermount-glusterfs':
> Operation not permitted
> /var/lib/jenkins/jobs/smoke/workspace/extras/geo-rep/gsync-sync-gfid.c:1
> 0:40: error: libglusterfs/src/glusterfs.h: No such file or directory
> /var/lib/jenkins/jobs/smoke/workspace/extras/geo-rep/gsync-sync-gfid.c:1
> 1:38: error: libglusterfs/src/syscall.h: No such file or directory
> cc1: warnings being treated as errors
> /var/lib/jenkins/jobs/smoke/workspace/extras/geo-rep/gsync-sync-gfid.c:
> In function 'main':
> /var/lib/jenkins/jobs/smoke/workspace/extras/geo-rep/gsync-sync-gfid.c:9
> 0: error: implicit declaration of function 'sys_lsetxattr'
> make[2]: *** [gsync_sync_gfid-gsync-sync-gfid.o] Error 1
> make[1]: *** [install-recursive] Error 1
> make: *** [install-recursive] Error 1
> Build step 'Execute shell' marked build as failure
> Finished: FAILURE
>
>
I just faced a similar issue while building the upstream code today 
because I did not have the libattr-devel headers installed:

To build RPMS run 'make glusterrpms'
Making all in geo-rep
   CC       gsync_sync_gfid-gsync-sync-gfid.o
gsync-sync-gfid.c:7:24: fatal error: attr/xattr.h: No such file or directory
  #include <attr/xattr.h>
                         ^
compilation terminated.
make[3]: *** [gsync_sync_gfid-gsync-sync-gfid.o] Error 1
make[2]: *** [all-recursive] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2

Seems to be appearing because http://review.gluster.org/#/c/6926/4 was 
recently merged which compiles gsync-sync-gfid.c

Either install libattr-devel or do this:

diff --git a/extras/geo-rep/gsync-sync-gfid.c 
b/extras/geo-rep/gsync-sync-gfid.c
index 601f472..00c17b0 100644
--- a/extras/geo-rep/gsync-sync-gfid.c
+++ b/extras/geo-rep/gsync-sync-gfid.c
@@ -4,7 +4,7 @@
  #include <string.h>
  #include <limits.h>
  #include <sys/types.h>
-#include <attr/xattr.h>
+#include <sys/xattr.h>
  #include <libgen.h>
  #include <ctype.h>
  #include <stdlib.h>

Not sure which one is better :)