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

support /etc/pexrc #183

Closed
wants to merge 1 commit into from
Closed

Conversation

landism
Copy link
Contributor

@landism landism commented Nov 19, 2015

This will allow system administrators to make it more likely that pexes are executed using a "clean" python instead of whatever users have installed on their path. /etc/pexrc is the first rc read so that it has the lowest precedence and determined users can still override whatever they like.

@kwlzn
Copy link
Contributor

kwlzn commented Nov 19, 2015

merged @ 62ffd3b via the rbt workflow. closing.

@kwlzn kwlzn closed this Nov 19, 2015
kwlzn pushed a commit to pantsbuild/pants that referenced this pull request Dec 2, 2015
Changelog:
```
* Fix infinite recursion when ``PEX_PYTHON`` points at a symlink.
  #182 pex-tool/pex#182

* Add ``/etc/pexrc`` to the list of pexrc locations to check.
  #183 pex-tool/pex#183

* Improve error messaging for platform constrained Untranslateable errors.
  #179 pex-tool/pex#179
```

Testing Done:
https://travis-ci.org/pantsbuild/pants/builds/94480030

Bugs closed: 2654

Reviewed at https://rbcommons.com/s/twitter/r/3200/
peiyuwang pushed a commit to twitter/pants that referenced this pull request Dec 4, 2015
Changelog:
```
* Fix infinite recursion when ``PEX_PYTHON`` points at a symlink.
  pantsbuild#182 pex-tool/pex#182

* Add ``/etc/pexrc`` to the list of pexrc locations to check.
  pantsbuild#183 pex-tool/pex#183

* Improve error messaging for platform constrained Untranslateable errors.
  pantsbuild#179 pex-tool/pex#179
```

Testing Done:
https://travis-ci.org/pantsbuild/pants/builds/94480030

Bugs closed: 2654

Reviewed at https://rbcommons.com/s/twitter/r/3200/
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.

2 participants