-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Display the built file name instead of the canonicalized name in uv build
#11103
Comments
Your guesses are right: uv is parsing the filename and using its normalized string printer for it. This would always be the same value for a spec-compliant build backend, which setuptools isn't (pypa/setuptools#4766). We could warn about this but it's not something the user can fix (#8203). |
Why not display the unparsed file name? i.e., what does parsing / normalizing get us? |
I don't think we even have it. We probably store the |
Oh nevermind, we do get it back from the build backend. We should just show the "real" filename. |
The parsing part gives us a correctness check that the output file is valid and not an error. We can change the message to verbatim independent of that. |
uv build
do not necessarily match the "success" messageuv build
…build` (astral-sh#11593) We keep the raw filename (prior to parsing/normalization) on the `BuildMessage` so we can display it. Closes astral-sh#11103. Closes astral-sh#11635.
Summary
eg
giving output that ends
Successfully built ruamel_yaml-0.18.10-py3-none-any.whl
but this is not true, the setuptools backend is not PEP491 compliant, so the file that is actually built is
ruamel.yaml-0.18.10-py3-none-any.whl
.I guess uv is assuming that the output is always spec-compliant?
It's a pretty minor confusion.
(Also this particular example should go away soon, there's a pull request open in setuptools for PEP491-compliance that looks likely to merge any day now.)
Platform
ubuntu 24.04
Version
0.5.25
Python version
3.13
The text was updated successfully, but these errors were encountered: