Merge lp:~timo-jyrinki/compiz/migrate-profile-independent into lp:compiz/0.9.8
| Status: | Merged |
|---|---|
| Approved by: | Didier Roche on 2012-09-11 |
| Approved revision: | 3353 |
| Merged at revision: | 3368 |
| Proposed branch: | lp:~timo-jyrinki/compiz/migrate-profile-independent |
| Merge into: | lp:compiz/0.9.8 |
| Diff against target: |
248 lines (+103/-120) 4 files modified
postinst/convert-files/compiz-profile-Default.convert (+0/-60) postinst/convert-files/compiz-profile-active-Default.convert (+0/-60) postinst/convert-files/compiz-profile-independent-keys.convert (+59/-0) postinst/migration-scripts/03_migrate_profile_independent.py (+44/-0) |
| To merge this branch: | bzr merge lp:~timo-jyrinki/compiz/migrate-profile-independent |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Didier Roche | 2012-09-11 | Approve on 2012-09-11 | |
| jenkins (community) | continuous-integration | Approve on 2012-09-11 | |
|
Review via email:
|
|||
Commit Message
Migrate profile independent keys separately from the profile dependent keys.
Description of the Change
Migrate profile independent keys separately from the profile dependent keys. This is still needed to completely tackle LP: #1046190. We'll also need a patched gconf2 to support 's' -> 'as' conversions.
| Didier Roche (didrocks) wrote : | # |
Looking good, see my comment on IRC for changes in the packaging needed.
Other than that, just to be picky:
240 + if not client:
241 + print "WARNING: no gconf client found. No transitionning will be done"
242 + return
-> I would rather sys.exit(1) to mark it not transitionned yet.
- 3353. By Timo Jyrinki on 2012-09-11
-
return -> sys.exit(1)
| jenkins (martin-mrazik+qa) wrote : | # |
PASSED: Continuous integration, rev:3353
http://
Executed test runs:
SUCCESS: http://


PASSED: Continuous integration, rev:3352 jenkins. qa.ubuntu. com/job/ compiz- ci/66/ jenkins. qa.ubuntu. com/job/ compiz- ci/./build= pbuilder, distribution= quantal, flavor= amd64/66/ console
http://
Executed test runs:
SUCCESS: http://