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

Set LD_LIBRARY_PATH so Connext can find OpenSSL #647

Closed
wants to merge 1 commit into from

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented Mar 26, 2022

I'm not really sure what's supposed to be responsible for allowing Connext to find its custom OpenSSL libraries, but this seems to do the trick.

Note that sudo didn't seem to pass LD_LIBRARY_PATH through with -E, so I set it explicitly.

I'm not convinced that this is the best solution to this issue, but hopefully it can be used to guide us to a better one.

Before: Build Status
After: Build Status

I'm not really sure what's supposed to be responsible for allowing
Connext to find its custom OpenSSL libraries, but this seems to do the
trick.

Note that `sudo` didn't seem to pass LD_LIBRARY_PATH through with -E, so
I set it explicitly.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay added the bug Something isn't working label Mar 26, 2022
@cottsay cottsay requested a review from nuclearsandwich March 26, 2022 02:06
@cottsay cottsay self-assigned this Mar 26, 2022
@cottsay cottsay closed this Mar 28, 2022
@cottsay cottsay deleted the cottsay/connext-openssl branch March 5, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant