https://bugzilla.redhat.com/show_bug.cgi?id=961384

Stanislav Ochotnicky <socho...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |socho...@redhat.com

--- Comment #1 from Stanislav Ochotnicky <socho...@redhat.com> ---
Not doing a full review, just a few comments:
 * missing description
 * Group tag is not really used/needed anymore
 * Why do you need Requires(post[un]): vim? There are no post[un] scriptlets
 * %defattr macro is not needed either

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=9trfdM7HEw&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to