-
Notifications
You must be signed in to change notification settings - Fork 64
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
Use pip
's new --dry-run
and --report
flags
#321
Comments
Looks like https://pip.pypa.io/en/stable/reference/installation-report/ |
Should most of this support happen in |
I think so, yeah -- I figure we'll probably want a |
Reminder that the goal of |
Whoops, you're right. I misread |
Opened di/pip-api#153 on pip-api to track. |
FWIW, I wouldn’t bother with mirroring pip’s internals when using the installation report. pip-audit should be able to use the report format directly — when integrating into pip, it should be fairly straightforward to use the resolution report from within pip instead (or convert the objects into the relevant format). I’ll defer to you all for judging the right place to implement the subprocess call and JSON parse right now. :) The point of the report flag (along with dry run) is to make developing such pieces of functionality easier, outside of pip without needing to reinvent aspects of it. FWIW, there’s also pip inspect, which should allow replacing all the introspection logic for venvs with a subprocess call + JSON parse as well. |
Thanks @pradyunsg! Since the ultimate goal of Also, when integrating into |
This is done as of #523. |
On sufficiently new
pip
versions (not yet released), these flags will allow us to punt all dependency resolution directly topip
and retrieve a corresponding JSON report.pip-audit
should detect and support these versions ofpip
, leveraging their "official" dependency resolution flow rather than our internal one.See pypa/pip#10771.
The text was updated successfully, but these errors were encountered: