Skip to content

Feat/respect xdg spec on all os #2627

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Feat/respect xdg spec on all os #2627

wants to merge 9 commits into from

Conversation

KlassyKat
Copy link

This Pull Request fixes/closes #1498.

It changes the following:

  • Unifies OS logic for loading config.
  • Adds a ordered list of locations to search for config.
  • List includes XDG_CONFIG_HOME as the first priority.
  • Implements the same logic for getting the cache_path with XDG_CACHE_HOME as the top priority
  • No longer creates an empty gitui config folder by default.

I followed the checklist:

  • I added unittests
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog
@KlassyKat
Initial Implementation: Create a set of potential config locations to…
beb71fb 
… search for
@KlassyKat
Apply the same logic to get_app_cache_path
8e307d3
@KlassyKat
Refactor to make logic more reusable and testable
fb935b7
@KlassyKat
Realize I don't know how to spell candidates
819b4ca
@KlassyKat
refactor to be a little cleaner
cbc388d
@KlassyKat
don't create an empty config folder
299015a 
I am a bit hesitant about this change since I am unsure if any logic
depends on a config folder needing to exist. But as far as I can tell
nothing does and creating a empty config folder in someones dotfiles
when they don't want any configuration for gitui seems like terrible UX.
@KlassyKat
Add unit tests
9ea0d0d
@KlassyKat
Update docs to reflect new behavior
cde05b1
@KlassyKat
Add changelog item
e8440c2
@KlassyKat
Copy link
Author

KlassyKat commented Apr 29, 2025 •
edited
Loading

No longer creates an empty gitui config folder by default.

I'm a little trepidatious about this particular change since I am not certain there is no logic relying on a config folder existing; though as far as I can tell there isn't.

If it doesn't present any issues (or even if it does present minor issues) I am a strong advocate for not forcing an empty gitui config folder to exist for any users that don't feel any need to configure gitui. I see this as an extremely annoying UX for anyone that doesn't want/feel the need for any configuration of gitui and likes to keep their dotfiles clean.

For an app as functional as gitui out of the box I don't see the portion of users who don't feel the need for any additional config as a particularly small group.
KlassyKat
KlassyKat commented Apr 29, 2025
View reviewed changes
@KlassyKat
Copy link
Author

KlassyKat commented Apr 29, 2025 •
edited
Loading

Could also add a $GITUI_CONFIG_DIR env var to look for at the top of the priority list, but I see that as maybe a bit overkill for what is a relatively simple config structure.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@KlassyKat