Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent use of OPENSSL_CFLAGS & OPENSSL_LIBS in the build system #792

Closed
rgmiller opened this issue Aug 22, 2023 · 1 comment
Closed

Comments

@rgmiller
Copy link
Contributor

System information

Type Version/Name
Operating System Linux
OS Version Ubuntu 22.04
Architecture x86_64
UnifyFS Version git dev branch

Describe the problem you're observing

We are not consistent in our use of the OPENSSL_CFLAGS and OPENSSL_LIBS in our build system. Specifically, the configure script will set these variables in all the Makefiles, but only the Makefile for unifyfs-stage will actually use these variables. The practical upshot is that unifyfs-stage may end up being linked against a different version of libcrypto.

It's also noteworthy that if you're trying to use a non-default version of libcrypto (such as a version built by spack), unifyfs-stage will be the only part of UnifyFS that uses the requested version.

This is tangentially related to issue #723 and PR #791. Also note that these variable names might change as part of PR #791.

Describe how to reproduce the problem

To really see this, you probably want 2 versions of OpenSSL installed. (Having a spack-built version and a system-provided version is probably the easiest way to arrange this.) Then, after building UnifyFS, run ldd on unifyfs and unifyfs-stage. You should see that they link against two different versions of libcrypto.

Include any warning or errors or releveant debugging data

@CamStan
Copy link
Member

CamStan commented Aug 28, 2023

rgmiller added a commit to rgmiller/UnifyFS that referenced this issue Oct 6, 2023
Add OPENSSL_CFLAGS and OPENSSL_LIBS to the UNIFYFS_COMMON_BASE_FLAGS and
UNIFYFS_COMMON_BASE_LIBS variables, respectively.

Fixes issue LLNL#792
CamStan pushed a commit to CamStan/UnifyFS that referenced this issue Dec 20, 2023
Add OPENSSL_CFLAGS and OPENSSL_LIBS to the UNIFYFS_COMMON_BASE_FLAGS and
UNIFYFS_COMMON_BASE_LIBS variables, respectively.

Fixes issue LLNL#792
CamStan pushed a commit to CamStan/UnifyFS that referenced this issue Dec 20, 2023
Add OPENSSL_CFLAGS and OPENSSL_LIBS to the UNIFYFS_COMMON_BASE_FLAGS and
UNIFYFS_COMMON_BASE_LIBS variables, respectively.

Fixes issue LLNL#792
CamStan pushed a commit to CamStan/UnifyFS that referenced this issue Dec 20, 2023
Add OPENSSL_CFLAGS and OPENSSL_LIBS to the UNIFYFS_COMMON_BASE_FLAGS and
UNIFYFS_COMMON_BASE_LIBS variables, respectively.

Fixes issue LLNL#792
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants