Ticket #1774 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Add support for OLPC Powerd

Reported by: sayamindu Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: Unspecified by Release Team
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

Change History

Changed 3 years ago by sayamindu

Patch

Changed 3 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

Changed 3 years ago by sayamindu

  • status changed from new to closed
  • resolution set to fixed

Committed and pushed (fb2b76a90d350e0774ce878b3206cbf9c19a7bea)

Note: See TracTickets for help on using tickets.