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

Include zlib linker flag if not building with the vendored one #3524

Merged

Conversation

ahmadsherif
Copy link
Member

Apparently building with LLVM enabled includes this flag, probably because of using --system-libs, so building without LLVM fails because it can't link against zlib.

@yorickpeterse yorickpeterse self-assigned this Nov 16, 2015
@yorickpeterse
Copy link
Contributor

I restarted the OS X builds as they failed due to what appears to be an unrelated RubyGems/Ruby issue. Will check back again tomorrow to see the results.

@yorickpeterse
Copy link
Contributor

For some bizarre reason the build keeps failing on OS X when running gem --version. Since I don't use OS X I sadly can't test these changes locally, can anybody else using OS X run them to confirm everything is working fine?

@yorickpeterse
Copy link
Contributor

cc @brixen @jemc

@ahmadsherif
Copy link
Member Author

Looks like Ruby 2.0.0-p576 and RubyGems 2.5.0 are a bad combo on OS X, I think subsequent OS X builds will fail for the same reason. Can we bump Ruby version a little?

@jemc
Copy link
Member

jemc commented Nov 16, 2015

Sorry, no idea about OS X issues - I only build on linux.

@yorickpeterse
Copy link
Contributor

@ahmadsherif Bumping it to 2.2 should be fine, you're free to add this to this PR if you'd like (that way we can immediately see if it actually works).

OS X builds started to fail after the release of RubyGems 2.5.0, so this
should fix it.
Apparently building with LLVM enabled includes this flag, probably
because of using `--system-libs`, so building without LLVM fails because
it can't link against zlib.
@ahmadsherif
Copy link
Member Author

@yorickpeterse All passing except for the clang/linux job, which fails for the same reason as the clang/osx job from the last master build.

@yorickpeterse
Copy link
Contributor

@ahmadsherif Restarted build, it's green now. Thanks!

@yorickpeterse yorickpeterse merged commit b9ca4b3 into rubinius:master Nov 16, 2015
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.

None yet

3 participants