https://bugzilla.redhat.com/show_bug.cgi?id=839527 Peter Lemenkov <lemenkov@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #12 from Peter Lemenkov <lemenkov@xxxxxxxxx> --- (In reply to comment #11) > Thanks Peter. > > OK - changed to %_prefix/lib . > > Default on is intended, > > The service is not run as a daemon so no need for the subsys lock. > > New SPEC and SRPM: > > > SRPM: http://bsjones.fedorapeople.org/rtirq-20120505-4.fc17.src.rpm > SPEC: http://bsjones.fedorapeople.org/rtirq.spec Still missing LICENSE file in %doc - please add before importing. Otherwise looks good and I don't see any other issues. This package is APPROVED. ps it would be great if you review this one in return - https://bugzilla.redhat.com/652629 . Sorry it's rather big but I really desperately need this to be reviewed. -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review