Skip to content

Sync extconf.rb with ruby/ruby #523

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

Closed
wants to merge 1 commit into from
Closed

Conversation

peterzhu2118
Copy link
Member

@nobu
Copy link
Member

nobu commented Jul 8, 2022

This is kept un-backported intentionally, so that these warnings will be fixed.

@nobu
Copy link
Member

nobu commented Jul 9, 2022

I changed my mind.
It seems tough to “fix” warnings, until we drop the support for older OpenSSLs.
However since the current way is too compiler dependent, how about this?
https://github.com/nobu/openssl/tree/openssl3-warnings

@peterzhu2118
Copy link
Member Author

I think that looks good. But we should still sync the extconf.rb between the ruby/openssl and ruby/ruby right?

@nobu
Copy link
Member

nobu commented Jul 9, 2022

ruby/ruby@6d8f396 will be reverted and sync from here to ruby/ruby, in that case.

@hsbt
Copy link
Member

hsbt commented Nov 29, 2022

What's the next action? I'm not sure we choose nobu@ee5347a instead of this pull-request.

@hsbt
Copy link
Member

hsbt commented Nov 29, 2022

#576 is best way for this.

@hsbt hsbt closed this Nov 29, 2022
@hsbt hsbt deleted the pz-sync-extconf branch November 29, 2022 03:10
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