Code review comment for lp:~ploum-deactivatedaccount/serpentine/xdg_folders
Revision history for this message
| Tiago Cogumbreiro (cogumbreiro) wrote | # |
review:
Needs Resubmitting
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
> This patch changes the config dir from .serpentine to $XDG_CONFIG_HOME.
>
> As no critical informations are in the current .serpentine, there's no
> migration plan involved, I think it's a bit overkill to have a migration plan
> only for the playlist.xml.
In this project we try to make dependencies optional when feasible.
If the import of 'xdg.BaseDirectory' fails, then I would like the algorithm to fall back to the old method.
Could you do that?