https://bugzilla.redhat.com/show_bug.cgi?id=1301253
Brandon Thomas <bthoma...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bthoma...@gmail.com
--- Comment #1 from Brandon Thomas <bthoma...@gmail.com> ---
This is not an official review, and this does not effect the status of the
review request.
* It looks like a period slipped into the packages Summary.
* I don't believe that Amusements/Games" is a valid group.
* "make %{?_smp_mflags}" can be replaced with "%make_build".
* You don't need to specify the packager.
* You probably want the install and ln command to be in the "%install" section.
* You might want to consider including a change log at the end of your file.
--
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
