Merge lp:~didrocks/unity/fix-bug-655175 into lp:unity

Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 568
Proposed branch: lp:~didrocks/unity/fix-bug-655175
Merge into: lp:unity
Diff against target: 12 lines (+1/-1)
1 file modified
tools/migrate_favorites.py (+1/-1)
To merge this branch: bzr merge lp:~didrocks/unity/fix-bug-655175
Reviewer Review Type Date Requested Status
Mikkel Kamstrup Erlandsen (community) Approve
Review via email: mp+37609@code.launchpad.net

Description of the change

  replace = to _ for favorite migration (LP: #655175)

To post a comment you must log in.
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Approved!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tools/migrate_favorites.py'
2--- tools/migrate_favorites.py 2010-09-27 13:09:40 +0000
3+++ tools/migrate_favorites.py 2010-10-05 14:44:20 +0000
4@@ -37,7 +37,7 @@
5 def register_new_app(client, launcher_location, apps_list, priority_position):
6 key_name = 'app-%s' % launcher_location.split('/')[-1]
7 # remove forbidden key in gconf:
8- for val in [' ', '+', '\'', '"', '\\', '`', '[', ']', '(', ')', '°', '#', '~', '|', '&']:
9+ for val in [' ', '+', '\'', '"', '\\', '`', '[', ']', '(', ')', '°', '#', '~', '|', '&', '=']:
10 key_name = key_name.replace(val, '_')
11 # default distribution launcher don't begin with / and don't have a desktop file in ~/.gnome2
12 if os.path.exists(launcher_location) and key_name not in apps_list: