[LU-11160] build: implicit dependency on libuuid-devel Created: 19/Jul/18 Updated: 05/Aug/20 Resolved: 06/Aug/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.12.0, Lustre 2.10.6 |
| Type: | Bug | Priority: | Minor |
| Reporter: | CEA | Assignee: | Nathaniel Clark |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| 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 |
| Comments |
| Comment by Peter Jones [ 19/Jul/18 ] |
|
Nathaniel Can you please assist with this? Thanks Peter |
| Comment by James A Simmons [ 19/Jul/18 ] |
|
So I was right. We do need to add the dependency to lustre.spec.in and debain/control{.main} |
| Comment by Nathaniel Clark [ 19/Jul/18 ] |
|
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. |
| Comment by Nathaniel Clark [ 19/Jul/18 ] |
|
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. |
| Comment by Gerrit Updater [ 19/Jul/18 ] |
|
Nathaniel Clark (nclark@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/32842 |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32842/ |
| Comment by Peter Jones [ 06/Aug/18 ] |
|
Landed for 2.12 |
| Comment by Gerrit Updater [ 12/Sep/18 ] |
|
Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33147 |
| Comment by Gerrit Updater [ 04/Oct/18 ] |
|
John L. Hammond (jhammond@whamcloud.com) merged in patch https://review.whamcloud.com/33147/ |