https://bugzilla.redhat.com/show_bug.cgi?id=1273693 James Hogarth <james.hogarth@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@xxxxxxxxxxxxxxxxx |james.hogarth@xxxxxxxxx Flags| |fedora-review? --- Comment #2 from James Hogarth <james.hogarth@xxxxxxxxx> --- This would be useful for my work so I'm happy to do the review. Note I'm not a sponsor but following FESCO ticket 1499[1] I can carry out the review and once approved you can file a ticket at the sponsorship trac instance[2] indicating you need a sponsor for a package that has completed review. My initial pre-fedora-review comments: * Remove all that commented blurb at the top of the spec * Group is not required and shouldn't be used in current fedora specs * Use %autosetup macro rather than separate %setup then %patch macros * You are referencing %srcname which is not defined I'll add the more complete formal review within the next few days to add to the initial comments. [1] https://fedorahosted.org/fesco/ticket/1499 [2] https://fedorahosted.org/packager-sponsors/ -- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review