https://bugzilla.redhat.com/show_bug.cgi?id=996618
Ken Dreyer <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Assignee|[email protected] |[email protected] Flags| |fedora-review? --- Comment #1 from Ken Dreyer <[email protected]> --- I can take this review. While I work on the details, I noticed the following: 1. The %check section is not present. Can you add it? 2. I think %{gem_instdir}/README.rdoc belongs in the -doc subpackage. 3. Both %{gem_instdir}/MIT-LICENSE and %{gem_instdir}/README.rdoc should be marked as %doc. 4. I don't see why you would need to ship %{gem_instdir}/%{gem_name}.gemspec, since you're already shipping %{gem_spec}. In my own gems I always delete this second copy of the gemspec (although I can't find anything in the Ruby guidelines about this.) -- 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=fENjln3CR1&a=cc_unsubscribe _______________________________________________ package-review mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/package-review
