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

Replace $HOME/.xargo with $XDG_DATA_HOME/xargo #46

Closed
linkmauve opened this issue Sep 25, 2016 · 7 comments
Closed

Replace $HOME/.xargo with $XDG_DATA_HOME/xargo #46

linkmauve opened this issue Sep 25, 2016 · 7 comments

Comments

@linkmauve
Copy link

In order to respect the XDG Base Directory Specification, like cargo before it, xargo should switch to the standard $XDG_DATA_HOME/xargo directory (by default $HOME/.local/share/xargo when $XDG_DATA_HOME isn’t set).

@japaric
Copy link
Owner

japaric commented Sep 25, 2016

Cargo uses HOME not XDG_HOME_DIR AFAICT so I'd prefer not to differ from Cargo on this matter.

@linkmauve
Copy link
Author

They use $XDG_DATA_HOME/cargo currently, which is already better than yet another dotfile, and are in the process of moving their cache and configuration to the proper $XDG_CACHE_HOME/cargo and $XDG_CONFIG_HOME/cargo directories, respectively.

@severen
Copy link

severen commented Sep 28, 2016

Huh? Cargo still uses $HOME/.cargo for everything. I know this because I just installed and setup cargo yesterday.

@japaric
Copy link
Owner

japaric commented Sep 28, 2016

They use $XDG_DATA_HOME/cargo currently

@linkmauve I grepped for xdg in Cargo repository and there's only one match in a test, which actually unsets the env var.

My installation also uses ~/.cargo and there's @SShrike confirmation.

The "in the process" links you mention point to a RFC which hasn't been accepted.

I'd be happy to move Xargo to XDG after Cargo does it but I won't if Cargo doesn't.

I'm going to close this because it's not actionable until Cargo makes the change. Feel free to open a new issue by the time Cargo moves to XDG!

@japaric japaric closed this as completed Sep 28, 2016
Nils-TUD pushed a commit to Nils-TUD/xargo that referenced this issue Sep 17, 2019
Nils-TUD pushed a commit to Nils-TUD/xargo that referenced this issue Sep 17, 2019
46: Fix warning about implicit trait objects r=phil-opp a=phil-opp

Fixes japaric#45 

Co-authored-by: Philipp Oppermann <[email protected]>
@wackbyte
Copy link

Cargo supports setting the CARGO_HOME environment variable, though, so there should be a XARGO_HOME environment variable.

@RalfJung
Copy link
Collaborator

so there should be a XARGO_HOME environment variable.

@jakbyte is anything wrong with XARGO_HOME? I've been using it for years.

@wackbyte
Copy link

it took me a year to see this, but it does seem to work 🙂

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

No branches or pull requests

5 participants