Merge lp:~didrocks/unity/migrate-favorites into lp:unity
Proposed by
Didier Roche-Tolomelli
Status: | Merged |
---|---|
Merged at revision: | 489 |
Proposed branch: | lp:~didrocks/unity/migrate-favorites |
Merge into: | lp:unity |
Diff against target: |
132 lines (+111/-0) 3 files modified
tools/Makefile.am (+2/-0) tools/migrate_favorites.py (+95/-0) unity/unity-favorites.vala (+14/-0) |
To merge this branch: | bzr merge lp:~didrocks/unity/migrate-favorites |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mikkel Kamstrup Erlandsen (community) | Approve | ||
Review via email: mp+34858@code.launchpad.net |
Description of the change
To fix bug #622146, python script to migrate people from UNR launcher favorite, gnome panel and gnome desktop to unity launcher (non duplicate will be added to the panel).
The script is launched the first time the user is starting unity (it's a one time thing) from unity itself.
If the script fail, we don't care and continue loading unity
To post a comment you must log in.
Installation path: It looks like you install the migrate_ favorites. py script in $prefix/libexec? But in the vala code you call it in /usr/lib...
*.vala:
i) Can you use Config.PREFIXDIR or Config.PKGDATADIR or some other variable substitution for the script install path, otherwise you break jhbuild.
migrate_ favorites. py:
a) In get_desktop_dir(): The natural thing would be to use g_get_user_ special_ dir(), but it does not seem to be bound in Python... And I couldn't find it in the xdg.BaseDirectory or xdg.Config modules... So looks like you have to do what you are currently doing
b) Can you add a comment in the bottom of the script about "magical" flushing to gconfd? :-)
Approved provided you address i) and b) some way or other.