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

updated uv path determinatino with uv v0.5.0 installing to ~/.local/b… #4

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

mduncans
Copy link
Collaborator

…in/ instead of ~/.cargo/bin.

As of v0.5.0 of uv, uv is installed to ~/.local/bin instead of ~/.cargo/bin. This would cause issues in reportifyr for most functions. In this PR we introduce a get_uv_path() function to utils.R to determine if ~/.local/bin/uv or ~/.cargo/bin/uv exists and returns the path found with a preference to ~/.local/bin/uv if they both exist.

From https://github.com/astral-sh/uv/releases

  • Use XDG (i.e. ~/.local/bin) instead of the Cargo home directory in the installer (#8420)

Previously, uv's installer used $CARGO_HOME or ~/.cargo/bin for its target install directory. It's been a longstanding complaint that uv uses this directory, as there's no relationship to Cargo. Now, uv will be installed into $XDG_BIN_HOME, $XDG_DATA_HOME/../bin, or ~/.local/bin (in that order). Note that $UV_INSTALL_DIR can always be used to override the target directory. Upgrading with uv self update will not move uv to the new install directory.

@jacobdum jacobdum merged commit 1c08110 into main Nov 11, 2024
3 checks passed
@mduncans mduncans deleted the uv_path_update branch November 30, 2024 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants