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=638647 Rex Dieter <rdieter@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Blocks|177841(FE-NEEDSPONSOR) | Flag|fedora-review? |fedora-review+ --- Comment #34 from Rex Dieter <rdieter@xxxxxxxxxxxx> 2011-10-15 09:40:22 EDT --- thanks to all the pre-reviews... naming: ok license: ok macros: ok scriptlets: ok SHOULD: for f16+ anyway, would be nice to include native systemd support rather than legacy init script, see also: https://fedoraproject.org/wiki/Packaging:Systemd otherwise, fairly simple and clean, APPROVED. Wrt sponsorship, looks like you've already been included in packager group (sponsor rjones ), removing blocker. feel free to poke either of us if you have questions or need help with anything. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review