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

XDG compliance: ZINIT[HOME_DIR] no longer defaults to HOME/.zinit but to XDG_DATA_HOME/zinit #95

Merged
merged 3 commits into from
Nov 20, 2021

Conversation

pschmitt
Copy link
Member

@pschmitt pschmitt commented Nov 20, 2021

We updated the installer to install to XDG_DATA_HOME/zinit, but never updated zinit itself to actually look for its home dir in there.

Note that this also updates the default value ZINIT[HOME_DIR] if none of these paths exists to XDG_DATA_HOME/zinit (instead of HOME/.zinit).
Epic victory royale for the XDG gang.

Refs:

@pschmitt pschmitt added this to the zinit 4.0 milestone Nov 20, 2021
@pschmitt pschmitt changed the title Add XDG_DATA_HOME/zinit to the list of paths where ZINIT[HOME_DIR] might be XDG compliance: ZINIT[HOME_DIR] no longer defaults to HOME/.zinit but to XDG_DATA_HOME/zinit Nov 20, 2021
@alichtman
Copy link
Member

I patched this manually on my system last night and hadn't yet put up a PR. Thanks! XDG FTW

@alichtman alichtman merged commit 4904564 into master Nov 20, 2021
@alichtman alichtman deleted the xdg-zinit-home-dir branch November 20, 2021 13:54
@alichtman
Copy link
Member

We should add zinit to the Arch wiki list of XDG-compliant programs

@pschmitt pschmitt restored the xdg-zinit-home-dir branch November 30, 2021 10:57
@vladdoster vladdoster deleted the xdg-zinit-home-dir branch December 27, 2021 10:48
@github-actions
Copy link

github-actions bot commented Nov 7, 2022

🎉 This PR is included in version 3.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants