Skip to content

fix: cargo bin dir#49

Merged
hykilpikonna merged 2 commits intohykilpikonna:masterfrom
DaRacci:patch-1
Dec 7, 2022
Merged

fix: cargo bin dir#49
hykilpikonna merged 2 commits intohykilpikonna:masterfrom
DaRacci:patch-1

Conversation

@DaRacci
Copy link

@DaRacci DaRacci commented Dec 5, 2022

Use the environment variable to find the dir instead of the hard coded default

Use the environment variable to find the dir instead of the hard coded default
@hykilpikonna
Copy link
Owner

For some reason I don't have the environment variable $CARGO_HOME set up by default, but I do have the directory ~/.cargo/bin. It might be better to have a default fallback if $CARGO_HOME is not set (e.g. use "${CARGO_HOME:-$HOME/cargo}").

image

Co-authored-by: Hykilpikonna <me@hydev.org>
@hykilpikonna hykilpikonna merged commit 1b0cf4a into hykilpikonna:master Dec 7, 2022
@hykilpikonna
Copy link
Owner

Okay, it should work, thank you for the pull request!

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