Opened 13 years ago

Closed 13 years ago

#1774 closed enhancement (fixed)

Add support for OLPC Powerd

Reported by: sayamindu Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords: r+
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

The attached patch adds support for OLPC's powerd (​http://dev.laptop.org/git/users/pgf/powerd) in Sugar's Power control panel.

Attachments (1)

0001-Let-power-control-panel-auto-adapt-to-whether-powerd.patch​ (1.8 KB) - added by sayamindu 13 years ago.
Patch

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by sayamindu

Patch

comment:1 Changed 13 years ago by tomeu

  • Keywords r+ added; r? removed 
-
 import dbus
+import os
+import logging

Can you group the imports as per PEP-08?

+        return 0

I know the original code already did this, but it's confusing, can you remove the 0?

r+ with those comments addressed

comment:2 Changed 13 years ago by sayamindu

  • Resolution set to fixed
  • Status changed from new to closed

Committed and pushed (fb2b76a90d350e0774ce878b3206cbf9c19a7bea)

Note: See TracTickets for help on using tickets.