https://bugzilla.redhat.com/show_bug.cgi?id=989400
Ken Dreyer <ktdre...@ktdreyer.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ktdre...@ktdreyer.com --- Comment #1 from Ken Dreyer <ktdre...@ktdreyer.com> --- Hi Mamoru, Thanks for submitting the package. Here's a couple questions. 1. In %prep, I haven't seen this "tmpunpackdir" convention in many other gems, and I don't see it referenced in the Ruby packaging guidelines. If this convention is necessary for mini_portile, could you please add a comment to the .spec file indicating why this is necessary? 2. There is a section in %prep where you run "chmod 0644" on all the .rb files. From what I can see by running "gem unpack" by hand on mini_portile-0.5.1.gem, the single .rb file is already 644. In other words, it looks to me as if this step is unnecessary. Can you remove it from your .spec file? 3. From what I read in the current Ruby packaging guidelines, "gem build" is supposed to happen in %build. In your package, it happens in %prep. If there is a reason for this deviation, can you please clarify it in a comment in the .spec? 4. I'm curious about this line: %doc %{gem_instdir}/[A-Z]* Why not just specify LICENSE.txt explicitly here? From my understanding of the Ruby packaging guidelines, the History.txt and README.rdoc files belong in the -doc subpackage. 5. Regarding these lines: # Currently no useful %exclude %{gem_instdir}/examples/ The comment should read "Currently not useful". Also, can you please expand the comment to explain why it would not be useful to ship the example Rakefile in that directory? -- 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=twP2ZUZvBg&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review