-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Report the queried executable path in uv python list
#12628
Conversation
I can try to add some test coverage here, but it seems kind of tricky. |
@Gankra would you try this out on your bespoke Windows setup? |
Confirmed this is working the way you want:
|
Hmm wait does this change whether we actually invoke the shim? The "bug" was purely display? |
Can you say more? Interpreter discovery invokes each executable it finds. Previously, we reported |
The detail I was unclear on was whether we kept invoking through that executable once we found it, or if we cached the sys.executable path and used that directly. |
Well.. we keep invoking through |
In an attempt to avoid reporting shims as their resolved
sys.executable
path, we report the queried executable path instead. This seems more correct for this command, broadly? Interestingly, it changes the reported paths for Homebrew PythonCloses #9979