[Bug 507089] Review Request: olpc-powerd - power management for the XO laptop

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=507089


Christoph Wickert <fedora at christoph-wickert.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




--- Comment #8 from Christoph Wickert <fedora at christoph-wickert.de>  2009-07-22 20:37:59 EDT ---
One last thing I have spotted: We also need to require upstart for initctl:
# these are all for the manipulation of powerd
Requires(post): upstart
Requires(postun): upstart
Requires(preun): upstart


I have tested the package on my XO 1 powerd works fine, keys work etc.

test case for scriptlets:
$ /etc/init.d/ohmd status
$ rpm -Ivvh olpc-powerd-8-1.20090720git702fd8e.fc9.i386.rpm
$ /etc/init.d/ohmd status
$ ps -A | grep powerd

Upgrade:
$ rpm -Uvvh olpc-powerd-8-1.20090720git702fd8e.fc9.i386.rpm --force
$ ps -A | grep powerd
 (other process id)
$ /etc/init.d/ohmd status

Removal:
$ rpm -evv olpc-powerd
$ ps -A | grep powerd
 (other process id)
$ /etc/init.d/ohmd status

-> works as it should.

The package is APPROVED

... but please don't forget to add the upstart requires after import.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.