Software Updater

Merge lp://staging/~mterry/update-manager/requires-restart into lp://staging/update-manager

  1. requires-restart
  2. Merge into main
Proposed by Michael Terry
Status: Merged
Merged at revision: 2660
Proposed branch: lp://staging/~mterry/update-manager/requires-restart
Merge into: lp://staging/update-manager
Diff against target: 207 lines (+87/-20)
2 files modified
UpdateManager/UpdatesAvailable.py (+42/-15)
data/gtkbuilder/UpdateManager.ui (+45/-5)
To merge this branch: bzr merge lp://staging/~mterry/update-manager/requires-restart
Reviewer Review Type Date Requested Status
Michael Vogt (community) Approve
Review via email: mp+152723@code.staging.launchpad.net

Description of the change

Add label to warn the user when a selected update will need a restart.

From https://wiki.ubuntu.com/SoftwareUpdates#Presenting_installable_updates

(No packages currently have the right metadata -- so you'll have to manually edit pkg_requires_restart() to return True for a given package to test it.)

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

Looks good to me, nice use of "nonlocal "

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

hum, is there any reason that didn't get merged? it seems it needs to be rebased now, would be nice to get in for the LTS

lp://staging/~mterry/update-manager/requires-restart updated
2618. By Michael Terry

Merge from trunk

Revision history for this message
Michael Terry (mterry) wrote :

I think it didn't get in because it was too close to release at the time.

I've updated this again for trunk. Merge it in if you still like it!

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

to status/vote changes: