-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add missing openssl dependencies #5349
Conversation
Notifying maintainers: |
/cc @dbevans, who bumped p5-crypt-openssl-rsa |
Travis Build #8368 Errored. Lint results
Port qt5-qtbase success on xcode9.4. Log The build timed out. |
Rebased as py-cryptography is fixed independently in bd26908. |
Travis Build #8387 Errored. Lint results
Port qt5-qtbase's dependencies fail on xcode10.3. Log The build timed out. |
LGTM (for OpenIPMI) Thanks! |
Thank you for checking it out! @MarcusCalhoun-Lopez Any comment? Due to complexity of qt5 ports, I'd like to get advices from the maintainer before commiting any non-trivial changes. |
Thank you for looking into this. It seems that the problems with Qt Location are a bit more involved than just OpenSSL. May I humbly suggest that Qt be treated separately than the other two ports so as not to hold up this pull request? |
Whenever dependencies are changed, doesn't that necessitate a revbump? |
Usually, yes. There are cases when the port did not build where it is not necessary. |
${prefix}/lib/perl5/vendor_perl/5.28/darwin-thread-multi-2level/auto/Crypt/OpenSSL/RSA/RSA.bundle links to lib{crypto,ssl}.1.1.dylib.
${prefix}/lib/libOpenIPMI.0.dylib and several other files links to libcrypto.1.1.dylib.
Done - I keep qt5 changes away from this pull request. I record the original commit here: b2c64723a6aebd46d395a0a9f49acc09998e79aa as its commit message might still be useful.
Thanks! I bump both remaining ports. |
Travis Build #8422 Failed. Lint results
Port p5.28-crypt-openssl-rsa success on xcode10.3. Log |
Port OpenIPMI fail on xcode7.3. Log
Well, this build fails as on the xcode7.3 builder (OS X 10.11),
version 2.0.19 is used, and that old version is not compatible with
OpenSSL 1.1.
@eborisch I think a decision is needed (use the openssl10 port or
backport patches from newer versions) before merging this PR.
|
Ugh. I'll try to talk a look at that when I have a chance. I doubt that the number of users on old systems using OpenIPMI is greater than 0 worldwide, so I may just have it error out. |
Got it! Then let's merge this first to benefit users on modern systems. |
Description
Some ports have files linked to libssl and/or libcrypto, but depends_lib does not include openssl, so they are missed in previous big OpenSSL upgrade (#3822). In this PR I fixed those ports to avoid further accidents.
Note that there might be more similar ports. These 4 are what I observe from commits to macports-ports after the big OpenSSL upgrade.
See individual commit messages for details, especially the one for qt*-qtlocation.
Type(s)
Tested on
Not tested
Verification
Have you
port lint
?sudo port test
?sudo port -vst install
?