https://bugzilla.redhat.com/show_bug.cgi?id=845568 Bohuslav "Slavek" Kabrda <bkabrda@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #2 from Bohuslav "Slavek" Kabrda <bkabrda@xxxxxxxxxx> --- - You should include Requires: ruby-libs, as it contains directory where the ext.so file is stored. If we ever have ruby(abi) also provided by JRuby or Rubinius, the file might end up useless and in an unowned directory (AFAICS it only works with MRI Ruby anyway). Otherwise, the package looks good, works ok and rpmlint is silent => APPROVED (please don't forget to add the Requires: as mentioned above). -- 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