Skip to content

Build dynamic library (libruby.so) #91

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

Merged
merged 2 commits into from
Dec 5, 2016

Conversation

aeriksson
Copy link

This PR adds libruby.so to the Ruby 2.1, 2.2, and 2.3 images by passing the --enable-shared flag to the ./configure call.

I ran into issues when trying to run a Ruby app with a native library that had to be linked against libruby.so in Docker. Since the Ruby base image don't have it, I got libruby.so.2.3: cannot open shared object file: No such file or directory. This seems like a use-case that should probably be supported by the Ruby Docker images. Thus this PR.

It seems to add a few megabytes to the image size (3.6, 3.4, 2.5 MB for Ruby 2.1, 2.2, and 2.3, respectively), but AFAICT it shouldn't have any adverse consequences otherwise.

@tianon
Copy link
Member

tianon commented Dec 5, 2016

Seems sane: https://sources.debian.net/src/ruby2.1/2.1.5-2%2Bdeb8u3/debian/rules/#L22

👍

I've pushed another commit to update the templates and thus all variants; as long as Travis comes back green, I think we're good to go! 🤘

@yosifkit yosifkit merged commit 50d97f7 into docker-library:master Dec 5, 2016
@tianon tianon mentioned this pull request Dec 5, 2016
@tianon
Copy link
Member

tianon commented Dec 5, 2016

docker-library/python#23 is further justification for this (beyond just that Debian enables this too)

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