Ubuntu
update-manager package

Merge lp://qastaging/~thekorn/ubuntu/lucid/update-manager/bug-539211-on_battery into lp://qastaging/ubuntu/lucid/update-manager

  1. Lucid (10.04)
  2. bug-539211-on_battery
  3. Merge into lucid
Proposed by Markus Korn
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp://qastaging/~thekorn/ubuntu/lucid/update-manager/bug-539211-on_battery
Merge into: lp://qastaging/ubuntu/lucid/update-manager
Diff against target: 35 lines (+15/-6)
1 file modified
UpdateManager/Core/utils.py (+15/-6)
To merge this branch: bzr merge lp://qastaging/~thekorn/ubuntu/lucid/update-manager/bug-539211-on_battery
Reviewer Review Type Date Requested Status
Michael Vogt Approve
James Westby (community) Approve
Review via email: mp+21396@code.qastaging.launchpad.net

Description of the change

Modified UpdateManager.Core.utils.on_battery() to use UPower per default
(and DeviceKit.Power as fallback) to check if a system is running on battery
(LP: #539211)

To post a comment you must log in.
Revision history for this message
James Westby (james-w) wrote :

Looks good to me.

Michael, would you review too please?

Thanks,

James

review: Approve
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks, this was merged a while ago:

update-manager (1:0.133.5) lucid; urgency=low

  [ Markus Korn ]
  * UpdateManager/Core/utils.py:
    - Modified UpdateManager.Core.utils.on_battery() to use UPower per default
      (and DeviceKit.Power as fallback) to check if a system is running
      on battery (LP: #539211)
...

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

to all changes: