-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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_CONFIG_HOME is not respected #6029
Comments
Hi, I wanted to contribute to resolving this issue and hence have raised a PR for this. Could you take a look at the PR please. Thank you. |
Config directory path in Linux should be picked from XDG_CONFIG_HOME environment variable. Fixes mumble-voip#6029
Thank you, I left a comment on the PR commit in question with some feedback. |
Config directory path in Linux should be picked from XDG_CONFIG_HOME environment variable. Fixes mumble-voip#6029
Config directory path in Linux should be picked from XDG_CONFIG_HOME environment variable Fixes mumble-voip#6029
Config directory path in Linux should be picked from XDG_CONFIG_HOME environment variable Fixes mumble-voip#6029
No need to comment here - I get notifications for anything that happens in the PR anyway |
Config directory path in Linux should be picked from XDG_CONFIG_HOME environment variable Fixes mumble-voip#6029
Ok, so I investigated this issue again. Mumble (at least the most recent versions) are respecting The respective code parts are: SQLite DB: Line 142 in a32229f
Config files: mumble/src/mumble/Settings.cpp Lines 1312 to 1314 in a32229f
The code OP posted originally is part of the fallback chain for legacy locations. These are prioritized in order from top to bottom and if a database is found at those locations, that will be used instead. The reason why OP ran into an issue is probably because they did not migrate The only issue remaining here is that some of the |
See #6459 |
Description
Currently the Mumble client hard-codes the location of the Mumble config directory to be located at ~/.config/mumble when it should read the XDG_CONFIG_HOME environment instead.
mumble/src/mumble/Database.cpp
Line 50 in f73db9d
Steps to reproduce
Mumble version
1.4.0
Mumble component
Client
OS
Linux
Reproducible?
Yes
Additional information
No response
Relevant log output
No response
Screenshots
No response
The text was updated successfully, but these errors were encountered: