Merge lp://qastaging/~mterry/ubuntu-release-upgrader/pkexec into lp://qastaging/ubuntu-release-upgrader

Proposed by Michael Terry
Status: Merged
Merged at revision: 2519
Proposed branch: lp://qastaging/~mterry/ubuntu-release-upgrader/pkexec
Merge into: lp://qastaging/ubuntu-release-upgrader
Diff against target: 622 lines (+428/-19)
13 files modified
DistUpgrade/DistUpgradeController.py (+0/-3)
DistUpgrade/DistUpgradeFetcher.py (+7/-7)
DistUpgrade/DistUpgradeViewGtk3.py (+0/-2)
DistUpgrade/GtkProgress.py (+97/-0)
data/com.ubuntu.release-upgrader.policy.in (+37/-0)
data/gtkbuilder/AcquireProgress.ui (+161/-0)
debian/ubuntu-release-upgrader-core.install (+2/-0)
do-partial-upgrade (+92/-0)
do-release-upgrade (+24/-5)
po/POTFILES.in (+4/-0)
pre-build.sh (+1/-1)
setup.cfg (+1/-0)
setup.py (+2/-1)
To merge this branch: bzr merge lp://qastaging/~mterry/ubuntu-release-upgrader/pkexec
Reviewer Review Type Date Requested Status
Serge Hallyn Approve
Ubuntu Core Development Team Pending
Review via email: mp+112638@code.qastaging.launchpad.net

Description of the change

This branch makes a few changes to make it easier to run pkexec when doing a release upgrade. It also separates out the code between update-manager and u-r-u slightly better.

This moves some code form u-m to here (that really should have been here, there will be a follow-on branch to drop that stuff from u-m).

It adds a policykit file to provide nice translated user strings when running pkexec on a u-r-u file.

It adds better support for running do-release-upgrade with a GTK frontend.

It adds a new command do-partial-upgrade, which does what update-manager does when it detected a partial upgrade is necessary. I've broken it out here into a new command so that it can more easily be recognized by pkexec and so because the code it used was all from DistUpgrade anyway.

To post a comment you must log in.
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

I see no obvious gotchas, looks good to me, thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches