[Bug 960056] Review Request: rubygem-debug_inspector - A Ruby wrapper for the MRI 2.0 debug_inspector API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #3 from Josef Stribny <jstribny@xxxxxxxxxx> ---
I am sorry for such a delay.

* Both %description for the package and its sub-package should end with a dot
(.)
* %{gem_instdir}/README.md should be in the main package since it contains
license (ideally you should ask upstream to split the license to a separate
file)
* %{gem_instdir}/README.md should be marked as %doc
* Why do you state # TODO: move the extensions if you are moving it already?

Otherwise the package works as expected.

-- 
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





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]