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

MSYS2 OpenSSL changed from 'etc/ssl' to 'bin/etc/ssl' #337

Closed
MSP-Greg opened this issue Mar 12, 2023 · 6 comments
Closed

MSYS2 OpenSSL changed from 'etc/ssl' to 'bin/etc/ssl' #337

MSP-Greg opened this issue Mar 12, 2023 · 6 comments

Comments

@MSP-Greg
Copy link
Contributor

Lars,

Another fun MSYS2 change. See msys2/MINGW-packages#16141.

Or, the following have all changed:

OpenSSL::X509::DEFAULT_CERT_FILE
OpenSSL::X509::DEFAULT_CERT_DIR
OpenSSL::Config::DEFAULT_CONFIG_FILE
@lazka
Copy link

lazka commented Mar 15, 2023

msys2/MINGW-packages#16141 wasn't supposed to change anything for the common case. Do you have any idea why this affects you?

@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented Mar 15, 2023

for the common case

The MSYS2 dll's included with Ruby are located in bin/ruby_builtin_dlls, so they are not accessible via PATH. This was done to alleviate problems with all the various Windows 'apps' that may share dll names.

Ruby's bin & exe files are in the bin folder. So, a long time ago, 'certs' were in the ssl folder, then moved to etc/ssl. With the recent change of the path being relative to the OpenSSL dll's path, it's now at bin/etc/ssl.

@lazka
Copy link

lazka commented Mar 15, 2023

uh, I see. I don't have a good idea on how to make both cases work :(

@MSP-Greg
Copy link
Contributor Author

Other than removing a trialing bin (or something similar) from the folder, I also don't see a good way of fixing it.

@larskanis
Copy link
Member

Thank you @MSP-Greg and @lazka for investigating this issue. I dithered between using a forked build script and accepting the new path, but decided to do the latter. It is a user visible change, since some users place their own certificates at the openssl path. They have to move it now. I hope it will be the last path change for the foresight future!

@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented Apr 1, 2023

I hope it will be the last path change for the foresight future

Strongly agree...

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

No branches or pull requests

3 participants