Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=472819
--- Comment #6 from Darryl L. Pierce <dpierce(a)redhat.com> 2009-04-29 16:19:55 EDT
---
(In reply to comment #5)
- I don't know if this is normal for ruby gems, but you're
using a non-standard
documentation location:
That's a standard location for Ruby gems:
(mcpierce@mcpierce-laptop:~)$ ls /usr/lib/ruby/gems/1.8/doc/
actionmailer-2.1.1 activeresource-2.1.1 daemons-1.0.7 gruff-0.3.4
rake-0.8.4 rufus-scheduler-1.0.13
actionpack-2.1.1 activesupport-2.1.1 fastthread-1.0.1 hoe-1.12.1
RedCloth-4.1.9 tlsmail-0.0.1
activerecord-2.1.1 cgi_multipart_eof_fix-2.3 gem_plugin-0.2.3 mongrel-1.0.1
rubyforge-1.0.3
Maybe the Ruby packaging guidelines should add a guideline for this?
http://fedoraproject.org/wiki/Packaging/Ruby#Build_Architecture_and_File_...
- The line
Requires: ruby(abi) = 1.8
is missing. Please add this. [
http://fedoraproject.org/wiki/Packaging/Ruby#Ruby_Packaging_Guidelines ]
Fixed.
- You can probably drop the
%define ruby_sitelib %(ruby -rrbconfig -e "puts
Config::CONFIG['sitelibdir']")
since you're not using it anywhere.
Removed.
SHOULD: If the package does not include license text(s) as separate
files from
upstream, the packager should query upstream to include it. NEEDSFIX
Fixed. LICENSE.txt is in the RPM but I didn't note it as a doc.
Spec URL:
http://mcpierce.fedorapeople.org/rpms/rubygem-rufus-scheduler.spec
SRPM URL:
http://mcpierce.fedorapeople.org/rpms/rubygem-rufus-scheduler-1.0.13-2.fc...
--
Configure bugmail:
https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.