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

Remove compat-openssl11 #884

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

neverpanic
Copy link
Contributor

We do not plan on supporting OpenSSL 1.1 in 10, so drop it from the package set. There should be no users of this in the distribution since we did not ship the development package.

@neverpanic
Copy link
Contributor Author

Actually, this is probably not correct, since it drops compat-openssl11 from the c9s label, but we'd only want to drop it from the eln label. Any specific guidance on how to do that?

@yselkowitz
Copy link
Collaborator

Split the configs into separate files, one for c9s and one for eln.

@neverpanic neverpanic force-pushed the crypto-no-openssl11 branch 2 times, most recently from 7e86e11 to 3222dc1 Compare June 26, 2023 12:57
@neverpanic
Copy link
Contributor Author

Split the configs into separate files, one for c9s and one for eln.

Thanks for the pointer. Done.

We do not plan on supporting OpenSSL 1.1 in 10, so drop it from the
package set for eln. There should be no users of this in the
distribution since we did not ship the development package.

Duplicate the configuration file so we can keep it for c9s.
@neverpanic
Copy link
Contributor Author

See also minimization/content-resolver#50

@bstinsonmhk bstinsonmhk merged commit 9d53a91 into minimization:main Jun 26, 2023
1 check passed
@neverpanic neverpanic deleted the crypto-no-openssl11 branch June 26, 2023 20:42
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.

3 participants