-
-
Notifications
You must be signed in to change notification settings - Fork 632
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
Add a centos6 Dockerfile #5167
Add a centos6 Dockerfile #5167
Conversation
a8c6dc2
to
f84795d
Compare
64d68b1
to
3654001
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm w a couple comments.
|
||
# TODO: Not https? Now we're really doing some science! | ||
RUN curl -o /etc/yum.repos.d/slc6-devtoolset.repo http://linuxsoft.cern.ch/cern/devtoolset/slc6-devtoolset.repo | ||
RUN rpm --import http://linuxsoft.cern.ch/cern/slc6X/x86_64/RPM-GPG-KEY-cern |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sketchy.. but not actually required, afaict? I'm able to yum install all of the below without this.
FROM centos:6 | ||
|
||
# Ensure Pants runs under the 2.7.13 interpreter. | ||
ENV PANTS_PYTHON_SETUP_INTERPRETER_CONSTRAINTS="['CPython==2.7.13']" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can always pass this in at runtime vs hardcoding in the image build, fwiw.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mm, yes. Will do that.
Problem
The wheels built in #5118 and consumed in #5159 are built in debian wheezy, meaning that they have a glibc 2.13 dependency... centos6 (still consumed in various places at Twitter) uses glibc 2.12.
More generally, it is beneficial to have binary releases of pants be compatible with as many linux distributions as possible (those that are in active use, at least).
Solution
Add a centos6 docker image, which was previously confirmed here to be capable of building pants.
Result
In a future change (once this image is published to https://hub.docker.com/r/pantsbuild/), we will switch the linux binary builder shard (
build-support/docker/travis_ci/Dockerfile
) to this image.