Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

ruby 2.0.0-p0 #18054

Closed
wants to merge 1 commit into from
Closed

ruby 2.0.0-p0 #18054

wants to merge 1 commit into from

Conversation

sferik
Copy link
Contributor

@sferik sferik commented Feb 24, 2013

This pull request is similar to #18052 but without the merge noise, a better commit message, and the correct sha256.

@sferik sferik mentioned this pull request Feb 24, 2013
@adamv
Copy link
Contributor

adamv commented Feb 24, 2013

Has 1.9 been submitted to homebrew-versions?

@sferik
Copy link
Contributor Author

sferik commented Feb 24, 2013

@mistydemeo
Copy link
Member

There's currently a discussion ongoing in rbenv/ruby-build#304 about whether this needs a conditional OpenSSL dependency. Need a little more investigation before it's pulled.

@mistydemeo
Copy link
Member

Looks like ruby needs to depend on a brewed openssl on Mountain Lion, but not on earlier OS Xs.

@adamv
Copy link
Contributor

adamv commented Feb 27, 2013

What's weird about the presumably newer Mtn Lion version?

@mistydemeo
Copy link
Member

I'm not sure. They're identical versions, and the symbol ruby checks for is available in both OSs. (The entirety of OpenSSL is also marked as deprecated in both Lion and Mt. Lion, so that's not different.) However, ruby doesn't find OpenSSL on Mt. Lion and does on Lion and lower.

@sferik
Copy link
Contributor Author

sferik commented Feb 27, 2013

I'm happy to squash these into a single commit. Just say the word.

@adamv
Copy link
Contributor

adamv commented Feb 27, 2013

Yes, please squash

@sferik
Copy link
Contributor Author

sferik commented Feb 27, 2013

Squashed. 🐛 👞

@jacknagel jacknagel closed this in d3455a1 Feb 27, 2013
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants