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

get-pipsi.py will now use the virtualenv installed in the python interpreter #91

Merged
merged 2 commits into from
Jun 25, 2017

Conversation

mmerickel
Copy link
Contributor

Instead of searching for the virtualenv command, use the virtualenv that
is installed in the python interpreter that is running
python get-pipsi.py. This will also fallback to using the venv
package on python3. It's assumed if someone installed virtualenv
then that overrides the default venv from the stdlib.

…rpreter

Instead of searching for the virtualenv command, use the virtualenv that
is installed in the python interpreter that is running
``python get-pipsi.py``. This will also fallback to using the ``venv``
package on python3. It's assumed if someone installed ``virtualenv``
then that overrides the default ``venv`` from the stdlib.
@mmerickel mmerickel force-pushed the bootstrap-with-venv branch from 0424d3e to be4abc4 Compare February 9, 2017 16:53
@mmerickel
Copy link
Contributor Author

Any chance this gets reviewed?

@RonnyPfannschmidt
Copy link
Contributor

looks good at first glance, could use an accompanying test

we are looking for a new maintainer

@mmerickel
Copy link
Contributor Author

@RonnyPfannschmidt I might be interested in helping with the maintenance of this project but I'd want the ability to push pypi releases as it hasn't had one in quite some time. It is not super clear to me right now what the ultimate goals are for this project versus how much it is considered feature-complete.

@mikepqr
Copy link

mikepqr commented Apr 30, 2017

Apart from the fact there's no tests for a pretty substantial change, this looks good to me.

pypi falling behind is a shame, and it would be great if a new maintainer had the ability to fix that, especially if there are no other maintainers. But given the recommended pipsi installation method, it's not the end of the world.

@RonnyPfannschmidt RonnyPfannschmidt merged commit 76a4ded into mitsuhiko:master Jun 25, 2017
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