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

Document that selfupdate requires call by python -m pip_review #47

Closed
ghost opened this issue Mar 31, 2017 · 4 comments
Closed

Document that selfupdate requires call by python -m pip_review #47

ghost opened this issue Mar 31, 2017 · 4 comments

Comments

@ghost
Copy link

ghost commented Mar 31, 2017

this problem happens for the same reason that pip needs to be upgraded with python -m upgrade pip. solutions for this are the following:

  • don't allow pip-review to upgrade itself
  • start a daemon process that waits until the parent has exited, then run the upgrade

@jgonggrijp what do you think?

@jgonggrijp jgonggrijp added the bug label Apr 4, 2017
@jgonggrijp jgonggrijp added this to the Version 1.0 milestone Apr 4, 2017
@jgonggrijp
Copy link
Owner

I'm hoping to merge and release #44 soon. Would that solve this issue? I would like to avoid both options in your bullet list, if possible.

@ghost
Copy link
Author

ghost commented Apr 4, 2017

No. This problem is specifically caused by entry_points (which I believe I added). The machinery that calls the function that you defined opens file locks that aren't returned until the entry point function returns. When pip is called, it notices those file locks and aborts.

@jgonggrijp
Copy link
Owner

If entry_points is the problem, then the entire problem can be avoided by calling python -m pip_review instead of the generated pip-review executable. Right?

FWIW, I implemented the entry_points solution.

@jgonggrijp jgonggrijp added question and removed bug labels Apr 5, 2017
@ghost
Copy link
Author

ghost commented Apr 5, 2017

If entry_points is the problem, then the entire problem can be avoided by calling python -m pip_review instead of the generated pip-review executable. Right?

Correct.

@jgonggrijp jgonggrijp changed the title windows: entry-point: pip-review cannot upgrade itself Document that selfupdate requires call by python -m pip_review Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant