Merge lp:~3v1n0/unity/migrate-to-sc-favorite into lp:unity
Proposed by
Marco Trevisan (Treviño)
on 2016-02-22
| Status: | Merged |
|---|---|
| Approved by: | Marco Trevisan (Treviño) on 2016-02-22 |
| Approved revision: | 4070 |
| Merged at revision: | 4075 |
| Proposed branch: | lp:~3v1n0/unity/migrate-to-sc-favorite |
| Merge into: | lp:unity |
| Diff against target: |
68 lines (+40/-2) 3 files modified
debian/unity.migrations (+1/-0) tools/migration-scripts/02_unity_setup_text_scale_factor (+3/-2) tools/migration-scripts/04_unity_update_software_center_desktop_file (+36/-0) |
| To merge this branch: | bzr merge lp:~3v1n0/unity/migrate-to-sc-favorite |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2016-02-22 | |
| Robert Ancell | 2016-02-22 | Approve on 2016-02-22 | |
|
Review via email:
|
|||
Commit Message
unity.migrations: add 04_unity_
Update favorite to use gnome software
To post a comment you must log in.
| Robert Ancell (robert-ancell) wrote : | # |
lp:~3v1n0/unity/migrate-to-sc-favorite
updated
on 2016-02-22
- 4070. By Marco Trevisan (Treviño) on 2016-02-22
-
04_unity_
update_ software_ center_ desktop_ file: more lines is better :)
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:4069
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
review:
Needs Fixing
(continuous-integration)


The tools/migration -scripts/ 02_unity_ setup_text_ scale_factor changes really should be a separate commit.
Please split:
favorites = [i.replace( 'ubuntu- software- center' , 'org.gnome. Software' ) if 'ubuntu- software- center. desktop' in i else i for i in favorites]
into multiple lines. It's really hard to understand.
Otherwise, it seems to do the right thing - migrate users from u-s-c to g-s.