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=518766


Rahul Sundaram <[email protected]> changed:

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




--- Comment #9 from Rahul Sundaram <[email protected]>  2009-08-27 16:32:17 
EDT ---
A few minor things:

Remove the # commented out lines. The description deosn't seem aligned and
seems too long. Unless you are branching out for EPEL, you don't need to define
the buildroot or remove it in the %install section anymore. 

IIUC, the mini scripts that are in /usr/share are really helper apps that
should be in /usr/libexec instead.

-- 
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
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to