[Bug 881751] Review Request: rubygem-pdf-inspector - A tool for analyzing PDF output

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

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=881751

--- Comment #2 from Bohuslav "Slavek" Kabrda <bkabrda@xxxxxxxxxx> ---
- rpmlint gives me:
"incorrect-fsf-address /usr/share/gems/gems/pdf-inspector-1.0.2/GPLv2", which
means that the LICENSE file is probably older. Although this is not a
showstopper, you should contact the upstream (or ideally send a pull request)
to fix this.
- Since this is a pure ruby Gem, it is not optimal to have runtime dependency
on Ruby. When we have more Ruby interpreters (e.g. JRuby), this gem should also
be usable with these. So please remove Requires: ruby, keeping only the runtime
requirement on ruby(abi).
- Please check the wordwrapping in description, it seems to be a little weird
:) (word "testing" is on its own line, etc.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
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]