-
Notifications
You must be signed in to change notification settings - Fork 20
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
Replace PyXDG module with appdirs #150
Conversation
`appdirs` returns appropriate platform-specific directories for saving user preferences and data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍
Please tell us, how you want to be listed in the |
Thanks! Mark Knoop mark@opus11.net |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before this pr, we used the same directory for the preferences.ini and the session xml files. Is it more correct to store the xml files in the data dir and the preferences.ini in the config dir?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than my above question, it looks good!
If we change the path for the We might then also need some kind of migration strategy. |
@SpotlightKid the PR as it is does look for and save xml files to the data dir and the preferences.ini goes to the config dir, where the xml files used to be. I tried this out and was surprised my session files weren't found. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My apologies. I retract all of my comments about behaviour changing with this PR. The locations of the xml files and preferences.ini remain the same. My bad!
appdirs
returns appropriate platform-specific directories for saving userpreferences and data.