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

Respect XDG base dir spec for config file #350

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

baodrate
Copy link

@baodrate baodrate commented Sep 8, 2021

Basic fix to use the XDG_CONFIG_HOME env var if it exists. Doesn't do more sophisticated platform-specific handling (it's an option but I'd personally hand that off to something like platformdirs).

closes #302

@baodrate
Copy link
Author

baodrate commented Sep 8, 2021

Logfile is currently set to {CONFIG_HOME}/error.log. It would be better to have this be {XDG_CACHE_HOME}/bpytop/error.log or something similar, but I'm not sure if it's worth changing.

@baodrate
Copy link
Author

baodrate commented Sep 8, 2021

Tried to update the documentation and default config file to indicate the new config directory. Examples use shell expansion to indicate behavior, suggestions for alternative formatting are welcome.

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.

[REQUEST] Respect XDG Base Dir Specification Environment Variables
1 participant