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



--- Comment #4 from Alexandre Beche <alexandre.be...@gmail.com> ---
It seems the package does not compile for EPEL5/6 so lets focus on ferora
release.

Before going through the main review, can I ask you the following small fixes:

- Put the license in a LICENSE file instead of the sources (not mandatory but
get the package a bit cleaner).

- Remove the commit number from the spec file (including the Release line and
the changelog).

- Clean the changelog (keep only the newest line properly formated as described
in [1]).

Cheers,
Alexandre

[1]:
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Changelogs

-- 
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
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to