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 <fed...@christoph-wickert.de> changed:

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




--- Comment #8 from Christoph Wickert <fed...@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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to