Bug 544964 - Review Request: rubygem-mime-types - Return the MIME Content-Type for a given filename
Summary: Review Request: rubygem-mime-types - Return the MIME Content-Type for a given...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 549286
TreeView+ depends on / blocked
 
Reported: 2009-12-07 07:16 UTC by Matthew Kent
Modified: 2014-07-21 12:39 UTC (History)
5 users (show)

Fixed In Version: 1.16-3.fc12
Clone Of:
Environment:
Last Closed: 2010-01-07 16:39:43 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Matthew Kent 2009-12-07 07:16:35 UTC
Spec URL: http://magoazul.com/wip/SPECS/rubygem-mime-types.spec
SRPM URL: http://magoazul.com/wip/SRPMS/rubygem-mime-types-1.16-1.fc13.src.rpm
Description: MIME::Types for Ruby manages a MIME Content-Type database that will return the Content-Type for a given filename.

MIME::Types was originally based on and synchronized with MIME::Types for Perl by Mark Overmeer, copyright 2001 - 2009. As of version 1.15, the data format for the MIME::Type list has changed and the synchronization will no longer happen.

mkent@fedora-devel-chef:~/rpmbuild/SPECS$ rpmlint rubygem-mime-types.spec /var/tmp/results/rubygem-mime-types-*
rubygem-mime-types-doc.noarch: W: no-documentation
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/like%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/ascii%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Types/%5b%5d-i.yaml %5b
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/binary%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/eql%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/signature%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/registered%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/obsolete%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/system%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/%3c%3d%3e-i.yaml %3c
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/docs%3d-i.yaml %3d
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/platform%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Type/complete%3f-i.yaml %3f
rubygem-mime-types-doc.noarch: W: misspelled-macro /usr/lib/ruby/gems/1.8/doc/mime-types-1.16/ri/MIME/Types/%5b%5d-c.yaml %5b
3 packages and 1 specfiles checked; 0 errors, 15 warnings.

Comment 1 Mamoru TASAKA 2009-12-23 19:41:21 UTC
Well,

* License
  - I guess the license for this package should
    be "GPL+ or Ruby or Artistic".

* rcov for %check
  - build.log says:
-------------------------------------------------------
    89  + rake test
    90  Hoe.new {...} deprecated. Switch to Hoe.spec.
    91  (in /builddir/build/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
    92  which: no dot in (/usr/sbin:/usr/bin:/sbin:/bin:/usr/X11R6/bin:/root/bin:/usr/local/sbin)
    93  /usr/bin/ruby -w -Ilib:ext:bin:test -e 'require "rubygems"; require "test/unit"; require "test/test_mime_type.rb"; require "test/test_mime_types.rb"' 
    94  RCov is not available. In order to run rcov, you must: sudo gem install spicycode-rcov
-------------------------------------------------------
    I don't know well about RCov, however when I install
    rubygem-rcov, rake check fails as:
-------------------------------------------------------
[tasaka1@localhost mime-types-1.16]$ rake test --trace
(in /home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
Hoe.new {...} deprecated. Switch to Hoe.spec.
rake aborted!
undefined method `test_files' for #<Hoe:0xb75d9a78>
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:53
/usr/lib/ruby/gems/1.8/gems/rcov-0.9.6/lib/rcov/rcovtask.rb:91:in `initialize'
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51:in `new'
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `load'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `raw_load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2016:in `load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in `standard_exception_handling'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2015:in `load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1999:in `run'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in `standard_exception_handling'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1997:in `run'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/bin/rake:31
/usr/bin/rake:19:in `load'
/usr/bin/rake:19
-------------------------------------------------------
  ref: http://bugs.gentoo.org/288996
  Would you check this?

Comment 2 Matthew Kent 2009-12-28 06:04:28 UTC
(In reply to comment #1)
> Well,
> 
> * License
>   - I guess the license for this package should
>     be "GPL+ or Ruby or Artistic".
> 

Sure! Fixed.

> * rcov for %check
>   - build.log says:
> -------------------------------------------------------
>     89  + rake test
>     90  Hoe.new {...} deprecated. Switch to Hoe.spec.
>     91  (in
> /builddir/build/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
>     92  which: no dot in
> (/usr/sbin:/usr/bin:/sbin:/bin:/usr/X11R6/bin:/root/bin:/usr/local/sbin)
>     93  /usr/bin/ruby -w -Ilib:ext:bin:test -e 'require "rubygems"; require
> "test/unit"; require "test/test_mime_type.rb"; require
> "test/test_mime_types.rb"' 
>     94  RCov is not available. In order to run rcov, you must: sudo gem install
> spicycode-rcov
> -------------------------------------------------------
>     I don't know well about RCov, however when I install
>     rubygem-rcov, rake check fails as:
> -------------------------------------------------------
> [tasaka1@localhost mime-types-1.16]$ rake test --trace
> (in
> /home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
> Hoe.new {...} deprecated. Switch to Hoe.spec.
> rake aborted!
> undefined method `test_files' for #<Hoe:0xb75d9a78>
> /home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:53
> /usr/lib/ruby/gems/1.8/gems/rcov-0.9.6/lib/rcov/rcovtask.rb:91:in `initialize'
> /home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51:in
> `new'
> /home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `load'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `raw_load_rakefile'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2016:in `load_rakefile'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in
> `standard_exception_handling'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2015:in `load_rakefile'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1999:in `run'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in
> `standard_exception_handling'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1997:in `run'
> /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/bin/rake:31
> /usr/bin/rake:19:in `load'
> /usr/bin/rake:19
> -------------------------------------------------------
>   ref: http://bugs.gentoo.org/288996
>   Would you check this?  

Rcov will produce a pretty report of the test coverage but is by no means necessary for the unit tests themselves. 

As to the error it looks like the Rakefile wants a specific version of rcov, 

p.extra_dev_deps  << %w(rcov ~>0.8)

while Fedora is on 0.9.6. I'll file a ticket with the upstream project about updating it to something more recent.

Comment 3 Matthew Kent 2009-12-28 06:18:04 UTC
Spec URL: http://magoazul.com/wip/SPECS/rubygem-mime-types.spec
SRPM URL: http://magoazul.com/wip/SRPMS/rubygem-mime-types-1.16-2.fc13.src.rpm

* Sun Dec 27 2009 Matthew Kent <mkent> - 1.16-2
- Fix license (#544964).
- Add note about rcov warning in test phase (#544964).

Comment 4 Mamoru TASAKA 2009-12-28 16:29:26 UTC
Well, what I am concerned here is that $ rake test actually
_fails_ is Fedora rubygem-rcov is installed (as I wrote in the
comment 2), not just warnings, so people who has rubygem-rcov installed
cannot rebuild your srpm, which I think should be avoided.

It seem that the following line
--------------------------------------------------
    53      t.test_files = hoe.test_files
--------------------------------------------------
in Rakefile seems unneeded. Would you again check this?

Comment 5 Matthew Kent 2009-12-30 08:46:05 UTC
(In reply to comment #4)
> Well, what I am concerned here is that $ rake test actually
> _fails_ is Fedora rubygem-rcov is installed (as I wrote in the
> comment 2), not just warnings, so people who has rubygem-rcov installed
> cannot rebuild your srpm, which I think should be avoided.
> 
> It seem that the following line
> --------------------------------------------------
>     53      t.test_files = hoe.test_files
> --------------------------------------------------
> in Rakefile seems unneeded. Would you again check this?  

Ah sorry, you're 100% right - I forget people may build without mock. I'll remove the line during the test phase.

Comment 6 Matthew Kent 2009-12-30 10:35:33 UTC
Seems the rcov task isn't necessary at all, newer versions of hoe provide the exact same thing in lib/hoe/rcov.rb.

Spec URL: http://magoazul.com/wip/SPECS/rubygem-mime-types.spec
SRPM URL: http://magoazul.com/wip/SRPMS/rubygem-mime-types-1.16-3.fc13.src.rpm

* Wed Dec 30 2009 Matthew Kent <mkent> - 1.16-3
- Remove needless rcov task in Rakefile causing issue (#544964).

Comment 7 Mamoru TASAKA 2009-12-30 17:42:01 UTC
One thing
- The part ">= 1.8.3" of "BR(check): rubygem(hoe)" is not needed
  unless you want to import this package into EPEL because
  rubygem-hoe packages in currently supported branches of Fedora all
  satisfy this version.

  ref: the last sentence of
  https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires
  (not a blocker)

-------------------------------------------------------------
  This package (rubygem-mime-types) is APPROVED by mtasaka
-------------------------------------------------------------

Comment 8 Matthew Kent 2009-12-31 00:33:31 UTC
(In reply to comment #7)
> One thing
> - The part ">= 1.8.3" of "BR(check): rubygem(hoe)" is not needed
>   unless you want to import this package into EPEL because
>   rubygem-hoe packages in currently supported branches of Fedora all
>   satisfy this version.
> 
>   ref: the last sentence of
>   https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires
>   (not a blocker)
>

Oops, thanks - missed that one.
 
> -------------------------------------------------------------
>   This package (rubygem-mime-types) is APPROVED by mtasaka
> -------------------------------------------------------------  

Thanks for the review.

Comment 9 Matthew Kent 2009-12-31 00:34:32 UTC
New Package CVS Request
=======================
Package Name: rubygem-mime-types
Short Description: Return the MIME Content-Type for a given filename
Owners: mkent
Branches: F-11 F-12
InitialCC:

Comment 10 Kevin Fenzi 2010-01-02 20:14:13 UTC
cvs done.

Comment 11 Fedora Update System 2010-01-07 07:27:30 UTC
rubygem-mime-types-1.16-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/rubygem-mime-types-1.16-3.fc11

Comment 12 Fedora Update System 2010-01-07 07:28:02 UTC
rubygem-mime-types-1.16-3.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/rubygem-mime-types-1.16-3.fc12

Comment 13 Mamoru TASAKA 2010-01-07 16:39:43 UTC
Closing.

Comment 14 Fedora Update System 2010-01-07 21:41:16 UTC
rubygem-mime-types-1.16-3.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2010-01-07 21:45:21 UTC
rubygem-mime-types-1.16-3.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 16 Michal Fojtik 2010-08-02 08:30:19 UTC
Hi, could you please push this package to F14/EL6 branch ?

Comment 17 Michael Stahnke 2010-09-10 18:14:07 UTC
Package Change Request
======================
Package Name: rubygem-mime-type
New Branches: el5 el6 
Owners: stahnma

Comment 18 Michael Stahnke 2010-09-10 18:39:23 UTC
Package Change Request
======================
Package Name: rubygem-mime-types
New Branches: el5 el6 
Owners: stahnma

Comment 19 Michael Stahnke 2010-09-10 18:40:07 UTC
mkent is aware of request.

Comment 20 Kevin Fenzi 2010-09-10 18:41:38 UTC
Git done (by process-git-requests).

Comment 21 Troy Dawson 2014-07-18 21:48:09 UTC
Package Change Request
======================
Package Name: rubygem-mime-types
New Branches: epel7
Owners: tdawson

Comment 22 Gwyn Ciesla 2014-07-21 12:39:42 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.