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

If on OS X, check for homebrew-supplied openssl libraries #3552

Merged
merged 2 commits into from Jan 6, 2016
Merged

If on OS X, check for homebrew-supplied openssl libraries #3552

merged 2 commits into from Jan 6, 2016

Conversation

@andre-richter
Copy link
Member

@andre-richter andre-richter commented Jan 6, 2016

No description provided.

@YorickPeterse
Copy link
Member

@YorickPeterse YorickPeterse commented Jan 6, 2016

@brixen Leaving this up to you since I think you were discussing something about this. I'm also not a Mac user so I'm not really sure what the context and the likes behind this is.

Loading

@andre-richter
Copy link
Member Author

@andre-richter andre-richter commented Jan 6, 2016

Hmm, travis timed out when building a native extension. Doesn't look like it has anything to do with my patch, though.

Loading

@YorickPeterse
Copy link
Member

@YorickPeterse YorickPeterse commented Jan 6, 2016

@andre-richter I restarted the build.

Loading

@brixen
Copy link
Member

@brixen brixen commented Jan 6, 2016

@andre-richter thanks for working on this. There's another place we check @darwin for the LLVM stuff. It's not critical that you do this, but could you take a little time to standardize both those checks? They are similar and having everything a bit more standard (ie consistent) will help when I port this to Bash soon.

Loading

@brixen
Copy link
Member

@brixen brixen commented Jan 6, 2016

@andre-richter thanks!

Loading

brixen added a commit that referenced this issue Jan 6, 2016
If on OS X, check for homebrew-supplied openssl libraries
@brixen brixen merged commit 982918f into rubinius:master Jan 6, 2016
1 check passed
Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants