[LU-11914] Build error for l_getsepol.c due to missing openssl/evp.h Created: 01/Feb/19 Updated: 24/Oct/23 Resolved: 10/May/19 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.13.0 |
| Fix Version/s: | Lustre 2.13.0, Lustre 2.12.2 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Andreas Dilger | Assignee: | Sebastien Buisson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | llnl | ||
| Environment: |
RHEL7.5 without openssl-devel |
||
| Issue Links: |
|
||||||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||||||
| Description |
|
I got a build failure due to a missing header for the newly-added l_getsepol command: l_getsepol.c:50:25: fatal error: openssl/evp.h: No such file or directory #include <openssl/evp.h> This requires the openssl-devel package to be installed in order to build, but that is not in the BuildRequires of lustre.spec file. Also, for sites that don't need this utility, the configure check should disable building l_getsepol if the header is not available. Finally, there is no l_getsepol.8 man page in lustre/doc for this command. |
| Comments |
| Comment by Sebastien Buisson [ 05/Feb/19 ] |
|
I will add a configure check that disables building l_getsepol in case openssl-devel is not available. But in this case, I do not see the point of adding a BuildRequires directive into the spec file: thanks to the configure check, there will be no hard dependency. |
| Comment by Gerrit Updater [ 05/Feb/19 ] |
|
Sebastien Buisson (sbuisson@ddn.com) uploaded a new patch: https://review.whamcloud.com/34183 |
| Comment by Sebastien Buisson [ 05/Feb/19 ] |
|
The man page for l_getsepol has been submitted under LU-930: |
| Comment by Gerrit Updater [ 27/Feb/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34183/ |
| Comment by Peter Jones [ 27/Feb/19 ] |
|
Landed for 2.13 |
| Comment by Olaf Faaland [ 08/May/19 ] |
|
Some of the SELinux related patches, such as
were backported to b2_12 for Lustre 2.12.1, but this patch was not, and so 2.12.1 has this build issue. |
| Comment by Olaf Faaland [ 08/May/19 ] |
|
The man page addition also did not get backported to 2.12.1, I believe. |
| Comment by Gerrit Updater [ 08/May/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34832 |
| Comment by Gerrit Updater [ 10/May/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34832/ |
| Comment by Peter Jones [ 10/May/19 ] |
|
I think that we set now |