[LU-8028] build fails after configure --disable-modules

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.11.0
• Type: Bug
• Priority: Minor
• Reporter: Robert Read (Inactive)
• Assignee: Dmitry Eremin (Inactive)
• Resolution: Fixed
• Severity: 3

Description

This is on current master, CentOS 7.

$ sh autogen.sh && ./configure --disable-modules
...
$ make
...
gcc -DHAVE_CONFIG_H -I. -I../..  -D_GNU_SOURCE -D_LARGEFILE64_SOURCE=1 -D_FILE_OFFSET_BITS=64 -DLUSTRE_UTILS=1 -include /work/lustre-release/undef.h -include /work/lustre-release/config.h -I/work/lustre-release/libcfs/include -I/work/lustre-release/lnet/include -I/work/lustre-release/lustre/include  -fPIC -g -O2 -Wall -Werror -MT libcfsutil_a-parser.o -MD -MP -MF .deps/libcfsutil_a-parser.Tpo -c -o libcfsutil_a-parser.o `test -f 'util/parser.c' || echo './'`util/parser.c
In file included from <command-line>:0:0:
/usr/include/stdc-predef.h:40:1: fatal error: /work/lustre-release/undef.h: No such file or directory
 #endif
 ^
compilation terminated.make[4]: *** [libcfsutil_a-parser.o] Error 1
make[4]: Leaving directory `/work/lustre-release/libcfs/libcfs'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/work/lustre-release/libcfs/libcfs'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/work/lustre-release/libcfs'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/work/lustre-release'
make: *** [all] Error 2

The workaround is to do make undef.h.

Comments

[Dmitry Eremin (Inactive)]

The dependency from undef.h is specified for modules and dist-hook only. So, just need to add it to all-am. Will push patch soon.

[Gerrit Updater]

Dmitry Eremin ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30965
Subject: LU-8028 build: fix make dependencies for --disable-modules
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 17d50a8be07a1ef280fd07267e8ded951157c4be

[James A Simmons]

Its really that easy to fix? This is a good step to getting the lustre utils with the upstream tree.

[Dmitry Eremin (Inactive)]

yes, this is easy fix for this issue.  You can check that it helps.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30965/
Subject: LU-8028 build: fix make dependencies for --disable-modules
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: be77f4b1a5d064a4eb5c96008598c1912fe81bef

[Peter Jones]

Landed for 2.11