[LU-13855] build: missing dependency check on binutils-devel

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Affects Version/s: Upstream
• Severity: 3
• Assignee: Neil Brown
• Reporter: Quentin Bouget (Inactive)

Description

Trying to compile the latest version of Lustre on Centos7 failed with:

# bash autogen.sh && rpm --eval %configure | bash && make -j 16
(...)
gcc -DHAVE_CONFIG_H -I. -I../..   -include /home/user/lustre-release/undef.h -include /home/user/lustre-release/config.h -I/home/user/lustre-release/lnet/include/uapi -I/home/user/lustre-release/lustre/include/uapi -I/home/user/lustre-release/libcfs/include -I/home/user/lustre-release/lnet/utils -I/home/user/lustre-release/lustre/include  -fPIC -D_GNU_SOURCE -D_LARGEFILE64_SOURCE=1 -D_FILE_OFFSET_BITS=64 -O2 -g -Wall -Werror -c -o statmany.o statmany.c
statx.c:53:10: fatal error: libiberty.h: No such file or directory
 #include <libiberty.h>
          ^~~~~~~~~~~~~
compilation terminated.

Installing binutils-devel fixes the issue.

Comments

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40802/
Subject: LU-13855 tests: remove the need for libiberty
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8b02ade2818d86f29965dae61918456330f38d55

[Andreas Dilger]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40802
Subject: LU-13855 tests: remove the need for libiberty
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 46ed9ccdefe3c89161ef2df3dbe3b5bf844b1707

[Quentin Bouget (Inactive)]

Thanks!

[Andreas Dilger]

I'm not sure where the LU ticket number came from, but the following patch landed to master, and probably should have been under this LU ticket:

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/39749/
Subject: LU-13902 config: add test for /usr/include/libiberty/
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e5923357b4ec5177e8cc540c5603f0f9df41de1e

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/39653
Subject: LU-13855 build: Move GLIBC check to where they are needed.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0698ca2070d1ff6a7c505db4bb3002b077eb1258

[Quentin Bouget (Inactive)]

I mean master.

[Peter Jones]

By "latest version" do you mean master or b2_12?