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

SPACESHIP_CONFIG_PATH not set correctly #1184

Closed
2 tasks done
ksalman opened this issue Sep 5, 2022 · 4 comments
Closed
2 tasks done

SPACESHIP_CONFIG_PATH not set correctly #1184

ksalman opened this issue Sep 5, 2022 · 4 comments
Labels
bug Bug related to code base, behavior, displaying, etc. released Released issues and PRs

Comments

@ksalman
Copy link

ksalman commented Sep 5, 2022

Is there an existing issue for this?

  • Yes, I have searched the existing issues
  • Yes, I have checked troubleshooting page

The problem

I created a config file at $HOME/.config/spaceship/spaceship.zsh but it is not loaded. It looks like the path is not loaded correctly here:
https://github.com/spaceship-prompt/spaceship-prompt/blob/master/lib/config.zsh#L11

Instead of:
"${XDG_CONFIG_HOME:="$HOME/.config/spaceship"}/spaceship.zsh"

It should be:
"${XDG_CONFIG_HOME:="$HOME/.config"}/spaceship/spaceship.zsh"

My $XDG_CONFIG_HOME is /Users/ksalman/.config

Here is how the array looks like with the current code. The 3rd and 4th elements are the same

➜ echo $SPACESHIP_CONFIG_PATH
/Users/ksalman/.spaceshiprc /Users/ksalman/.spaceshiprc.zsh /Users/ksalman/.config/spaceship.zsh /Users/ksalman/.config/spaceship.zsh /etc/xdg/spaceship.zsh

Relevant Zsh configuration

irrelevant

What is version of Spaceship are you using?

4.2.1

What is version of Zsh are you using?

zsh 5.8 (x86_64-apple-darwin20.0)

What operation system are you running?

macOS

Do you use any Zsh framework or plugin manager?

No response

Which terminal emulator do you use?

iTerm

Additional information

No response

@ksalman ksalman added the bug Bug related to code base, behavior, displaying, etc. label Sep 5, 2022
@denysdovhan
Copy link
Member

Good catch! Would you like to send a PR?

@ksalman
Copy link
Author

ksalman commented Sep 5, 2022 via email

github-actions bot pushed a commit that referenced this issue Sep 5, 2022
## [4.2.4](v4.2.3...v4.2.4) (2022-09-05)

### Bug Fixes

* **config:** fix wrong path for spaceship config ([cf4b55e](cf4b55e)), closes [#1184](#1184)
@denysdovhan
Copy link
Member

🎉 This issue has been resolved in version 4.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@denysdovhan denysdovhan added the released Released issues and PRs label Sep 5, 2022
@ksalman
Copy link
Author

ksalman commented Sep 5, 2022

Thanks for fixing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug related to code base, behavior, displaying, etc. released Released issues and PRs
Development

No branches or pull requests

2 participants