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 OpenSSL link path for macOS at install time #21

Closed
jborean93 opened this issue Feb 15, 2021 · 0 comments · Fixed by #25
Closed

Set OpenSSL link path for macOS at install time #21

jborean93 opened this issue Feb 15, 2021 · 0 comments · Fixed by #25

Comments

@jborean93
Copy link
Owner

jborean93 commented Feb 15, 2021

SUMMARY

Right now the macOS library is hardcoded to link to /usr/local/opt/openssl@1.1/lib/libssl.1.1.dylib and /usr/local/opt/openssl@1.1/lib/libcrypto.1.1.dylib for the OpenSSL libs. This path is the one used by the x86_64 brew installer when installing the openssl package. Usually this is fine for most people but there are some new cases popping up when this might not be desirable like

  • People using MacPorts to install OpenSSL, uses a different location
  • People compiling their own OpenSSL version to a custom location
  • When PowerShell eventually creates an arm64 port then the arm64 OpenSSL libs from brew are in a different location

What I'm hoping to implement is for Install-WSMan to check the various known locations of the OpenSSL libraries and select the one relevant to the architecture that PowerShell is running on. It will then use install_name_tool to rewrite the linked path to the libraries selected when it is installed making it work by default in more locations.

We could even add a check/warning to indicate when the wrong OpenSSL architecture was installed or no OpenSSL libs were found making it easier for end users to figure out why WSMan isn't working rather than the cryptic failed to load WSMan libraries error from PowerShell.

Related #20

LIBMI VERSION

N/A

OS / ENVIRONMENT

macOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant