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

Minor false positive warning for uvx reinstall #6576

Closed
bluss opened this issue Aug 24, 2024 · 3 comments · Fixed by #6609
Closed

Minor false positive warning for uvx reinstall #6576

bluss opened this issue Aug 24, 2024 · 3 comments · Fixed by #6609
Assignees
Labels
error messages Messaging when something goes wrong

Comments

@bluss
Copy link
Contributor

bluss commented Aug 24, 2024

Original command:

uvx --reinstall --with dist/*.whl pytest

Modified command (to try to avoid the warning):

uvx --reinstall-package my-package-name --with dist/*.whl pytest     

Both have the following warning. Especially in the second modified case, this is a false positive? Because the reinstall is wanted for the extra dependency added with --with.

warning: Tools cannot be reinstalled via `uvx`; use `uv tool upgrade --reinstall`
to reinstall all installed tools, or `uvx package@latest` to run the latest
version of a tool

The command is a recipe for running tests vs the freshly built wheel -- from the project directory. Reinstall to ensure the new wheel is used even if version etc did not change.
Using uv 0.3.3.

@charliermarsh
Copy link
Member

I think --refresh might be intended here.

@bluss
Copy link
Contributor Author

bluss commented Aug 24, 2024

Oh, thanks, I think refresh works for the use case, looks like it. I've just been keen to ensure the wheel is actually reinstalled. (With that fix, there is no issue here).

@charliermarsh
Copy link
Member

I think the warning needs to be tweaked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
error messages Messaging when something goes wrong
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants