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

Add the ability to build a pex to the release script #5159

Merged
merged 2 commits into from
Dec 5, 2017

Conversation

stuhood
Copy link
Member

@stuhood stuhood commented Dec 5, 2017

Problem

We're building cross-platform wheels for pants and its dependencies in CI, but we don't yet have the ability to consume them to build a pex.

Solution

Add the ability to consume the cross platform wheels to build a pex.

Result

The resulting pex works on centos7 and osx, but does not yet work on centos6 due to older GLIBC. So the next steps will be to:

  1. adjust the linux binary builder shard to use an older distro
  2. add support for publishing a pex (somewhere) as part of the release process

@stuhood stuhood requested review from jsirois and kwlzn December 5, 2017 02:13
@stuhood stuhood mentioned this pull request Dec 5, 2017
Copy link
Member

@kwlzn kwlzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@stuhood stuhood merged commit 4537486 into pantsbuild:master Dec 5, 2017
@stuhood stuhood deleted the stuhood/build-pex branch December 5, 2017 17:02
stuhood pushed a commit that referenced this pull request Dec 13, 2017
### 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.
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