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 <rdie...@math.unl.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
             Blocks|177841(FE-NEEDSPONSOR)      |
               Flag|fedora-review?              |fedora-review+

--- Comment #34 from Rex Dieter <rdie...@math.unl.edu> 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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to