Skip to content

fix(install.sh): respect XDG_CONFIG_HOME#839

Merged
theRealPadster merged 2 commits intospicetify:mainfrom
Betelgeuse1:main
Sep 23, 2024
Merged

fix(install.sh): respect XDG_CONFIG_HOME#839
theRealPadster merged 2 commits intospicetify:mainfrom
Betelgeuse1:main

Conversation

@Betelgeuse1
Copy link
Copy Markdown
Contributor

@Betelgeuse1 Betelgeuse1 commented Sep 21, 2024

Apply the same logic that is used in
spicetify/cli/src/utils/path-utils.go:GetSpicetifyFolder.
$SPICETIFY_CONFIG_DIR is now, in order :

  • $SPICETIFY_CONFIG
  • $XDG_CONFIG_HOME/spicetify
  • $HOME/.config/spicetify

This fixes a bug when installing spicetify and marketplace with a custom $XDG_CONFIG_HOME, making the marketplace theme and custom app invisible to spicetify.

Apply the same logic that is used in
_spicetify/cli/src/utils/path-utils.go:GetSpicetifyFolder_.
$SPICETIFY_CONFIG_DIR is now, in order :
- $SPICETIFY_CONFIG
- $XDG_CONFIG_HOME/spicetify
- $HOME/.config/spicetify

This fixes a bug when installing `spicetify` and `marketplace` with a custom
$XDG_CONFIG_HOME, making the marketplace theme and custom app invisible to
spicetify.
@Betelgeuse1 Betelgeuse1 requested a review from a team as a code owner September 21, 2024 17:47
@Betelgeuse1 Betelgeuse1 requested review from CharlieS1103 and removed request for a team September 21, 2024 17:47
@theRealPadster theRealPadster merged commit 8d38e99 into spicetify:main Sep 23, 2024
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