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

Re-work native engine version. #4046

Closed
wants to merge 7 commits into from

Conversation

jsirois
Copy link
Contributor

@jsirois jsirois commented Nov 11, 2016

This switches the Native subsystem to load the proper native engine
library version from a resource file that is, in-turn, auto-managed by
the pants script itself ensuring that pants, both in-repo and as a
released package (pex or sdist or wheel), will always ask for the exact
native engine library version it was built and tested with.

A follow-up change will ensure we release a native engine library to
bintray for all packaged releases, fulfilling the released pants package
portion of the contract.

https://rbcommons.com/s/twitter/r/4367/

@jsirois
Copy link
Contributor Author

jsirois commented Nov 11, 2016

This is part of addressing #4035.

This switches the `Native` subsystem to load the proper native engine
library version from a resource file that is, in-turn, auto-managed by
the pants script itself ensuring that pants, both in-repo and as a
released package (pex or sdist or wheel), will always ask for the exact
native engine library version it was built and tested with.

A follow-up change will ensure we release a native engine library to
bintray for all packaged releases, fulfilling the released pants package
portion of the contract.
Things like isort now depend on a bootstrapped pants.
@jsirois
Copy link
Contributor Author

jsirois commented Nov 11, 2016

It also addresses #4034 and restores pants.pex use in ci.sh

@jsirois
Copy link
Contributor Author

jsirois commented Nov 11, 2016

Submitted @ cede1ec

@jsirois jsirois closed this Nov 11, 2016
@jsirois jsirois deleted the jsirois/issues/4035 branch November 11, 2016 22:08
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.

1 participant