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

openssl: Upgrade formula to v1.1.1u #805

Merged
merged 3 commits into from
Jul 20, 2023

Conversation

sevan
Copy link
Collaborator

@sevan sevan commented May 16, 2023

Build tested on Tiger PowerPC
With this change brew audit openssl has one error flagged which is Header files that shadow system header files were installed to "tigerbrew/Cellar/openssl/1.1.1t/include" but unsure how to address it. Pointers appreciated if this needs to be addressed before it can be merged.

@sevan
Copy link
Collaborator Author

sevan commented May 16, 2023

Looks like it does need to be addressed.

Resolves brew audit openssl failure regarding
"Header files that shadow system header files were installed to ..."
@sevan
Copy link
Collaborator Author

sevan commented May 17, 2023

Now just need need to clean up the fallout from the API change since OpenSSL 1.1.0 for anything that's not up to date. :)

@sevan sevan changed the title openssl: Upgrade formula to v1.1.1t openssl: Upgrade formula to v1.1.1u May 30, 2023
@jjmarcos
Copy link

In case this helps anyone:
I just installed Tigerbrew on a G5 running Leopard.
It indicates that I should install git and curl after running brew doctor, which I tried to do but I found issues when building openssl.
I used the patched Formula for openssl 1.1.1i, but failed on testing.
Then I saw another note indicating that 1.1.1u was available, so I used Formula available here, thanks to it I was finally able to complete the build.

@goshhhy
Copy link

goshhhy commented Jul 13, 2023

i'm seeing the same behavior - TigerBrew does not work out of the box due to test failures in OpenSSL which prevent the installation of curl and git. manually installing this recipe works around the issue successfully.

@mistydemeo
Copy link
Owner

Building now. Given the number of other packages that will need to be rebuilt, I'll try to coordinate merges.

@mistydemeo mistydemeo changed the base branch from master to openssl_1_1 July 20, 2023 17:58
@mistydemeo mistydemeo merged commit a43062a into mistydemeo:openssl_1_1 Jul 20, 2023
@mistydemeo
Copy link
Owner

Merged into the OpenSSL 1.1 branch.

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 this pull request may close these issues.

4 participants