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

Miroslav Suchý <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |177841 (FE-NEEDSPONSOR)



--- Comment #2 from Miroslav Suchý <[email protected]> ---
Few comments:
* please use %{buildroot} instead of $RPM_BUILD_ROOT so you use macros
consistently.
* summary should not have trailing dot
  https://fedoraproject.org/wiki/Packaging:Guidelines#summary
* since you are the author of this tool, can you provide license file and put
it in %files section as %license
  https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text

Since this is your first package you need to block FE-NEEDSPONSOR as only
Fedora Sponsors can do your first package review (subsequent reviews can be
done by any packager).


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to