[LU-11160] build: implicit dependency on libuuid-devel

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.12.0, Lustre 2.10.6
• Severity: 3
• Assignee: Nathaniel Clark
• Reporter: CEA

Description

On centos7.4, Lustre now requires libuuid-devel to be installed, the configure script should check for it.

libtool: link: gcc -fPIC -D_GNU_SOURCE -D_LARGEFILE64_SOURCE=1 -D_FILE_OFFSET_BITS=64 -DLUSTRE_UTILS=1 -g -O2 -Wall -Werror -o llverdev llverdev.o  -luuid
/usr/bin/ld: cannot find -luuid

Issue Links

• is related to: LU-11097 undefined reference to `uuid_unparse@UUID_1.0'

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/33147/
Subject: LU-11160 build: Fix uuid / blkid dependency
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: c4116b86130a4bffacb145f4eedbd2ed6a567495

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33147
Subject: LU-11160 build: Fix uuid / blkid dependency
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: de6ee3d9ab77f485b72bb01b9dc0baff24fa4964

[Peter Jones]

Landed for 2.12

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/32842/
Subject: LU-11160 build: Fix uuid / blkid dependency
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b6c2ffa661f5aceb50fe03fbc8cfd7cdf7966ae2

[Gerrit Updater]

Nathaniel Clark ([e-mail]) uploaded a new patch: https://review.whamcloud.com/32842
Subject: LU-11160 build: Fix uuid / blkid dependency
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f140f0a235f5c9dfcb60f3e70ad2c1a12755651c

[Nathaniel Clark]

For el7.4 (and el7.5):

libblkid requires libuuid

libblkid-devel requires pkgconfig(uuid) which is provided by libuuid-devel

So "-luuid" should only be present if "-lblkid" is present.  I will submit a patch.

llverdev doesn't require libblkid when being built, it's an optional build dependency.

[Nathaniel Clark]

With regards to libuuid, I think it's actually a dependency of libblkid

So the following patch:

https://review.whamcloud.com/32726

may not be 100% correct.

[James A Simmons]

So I was right. We do need to add the dependency to lustre.spec.in and debain/control{.main}

[Peter Jones]

Nathaniel

Can you please assist with this?

Thanks

Peter