Use appdirs module for platform specific best practices regarding file locations #36
Conversation
Uses the new module 'tools.dirs' to retrieve absolute paths of all directories the application can write to. The variables of these folders are quite self-explanatory. Usage: path = dirs.data + "/myfile.txt"
|
Executing the dirs.py results in the following output on Windows 10 x64 Build 14393: For some reason, appdirs.appname and appdirs.appauthor are not included in any filepath. |
| os.chdir(project_home) | ||
|
|
||
| _setup_logging() | ||
| _setup_logging(file=False) |
There was a problem hiding this comment.
Logging to file should be on by default or atleast be configurable (and the config option should be turned on by default). This would make it easier to report crash reports for users who only use the gui option (and can not see the stracktrace in the commandline).
There was a problem hiding this comment.
I agree, however I didn't make any changes to the default here. This is the behavior we had before, and that is a seperate issue.
That is because I was doing it wrong. Should be fixed now though. |
Use appdirs module for platform specific best practices regarding file locations
Should be quite self explanatory. Fixes #20.
If there is anything missing, please let me know. If not, I think we can merge this rather quickly.