Bug 967332 - Review Request: rubygem-chardet - Character encoding auto-detection in Ruby
Review Request: rubygem-chardet - Character encoding auto-detection in Ruby
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Vít Ondruch
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2013-05-26 13:23 EDT by Anuj More
Modified: 2016-01-04 03:59 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2016-01-04 03:59:38 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Anuj More 2013-05-26 13:23:44 EDT
Spec URL: http://anujmore.fedorapeople.org/pkgs/rubygem-chardet/rubygem-chardet.spec
SRPM URL: http://anujmore.fedorapeople.org/pkgs/rubygem-chardet/rubygem-chardet-0.9.0-1.fc19.src.rpm
Description: Character encoding auto-detection in Ruby
Fedora Account System Username: anujmore
Comment 1 Anuj More 2013-05-26 13:34:13 EDT
Builds on mock: https://raw.github.com/execat/Packages/master/rpmspecs/rubygem-chardet/mock
rpmlint does not complain: 
3 packages and 0 specfiles checked; 0 errors, 0 warnings.
Comment 2 Josef Stribny 2013-05-28 06:51:12 EDT
I will take it for a review.
Comment 3 Josef Stribny 2013-05-28 07:45:37 EDT
From your spec file:

# URL: http://blog.vava.cn/ # This URL is mentioned in the Gem but doesn't open
URL: https://github.com/janx/chardet

The original chardet gem is actually hosted on RubyForge[1] and maintained by Hui Zheng. The release is from 2006 and I don't see any new development or updates here.

The URL you are mentioning belongs to chardet2[2], which is pretty new, it's based on chardet and seems to be maintained.

Apart from that I am unsure whether is chardet Ruby 2.0 ready or not, since I can see that there is a commit regarding Ruby 2.0 compatibility in chardet2[3]. Fedora 19 will use Ruby 2.0 as a default and it would be pointless to package a sort-of dead project that wasn't tested on Ruby 2.0 (although backporting of the above mentioned commit could solve it).

Is there a reason to package this gem? Wouldn't be better to bring chardet2 to Fedora?

[1] http://rubyforge.org/projects/chardet/
[2] https://rubygems.org/gems/chardet2
[3] https://github.com/janx/chardet2/commit/32cc6147b2e519eb3bd777e0bc854b5bfac11ecd
Comment 4 Anuj More 2013-05-28 08:54:39 EDT
I discussed this with the maintainer yesterday:

The change in URL and the repo name was done less than 10 days ago, where as I packaged the Gem quite some time ago.

After a discussion with vondruch, I packaged this Gem again as chardet2, and it worked with no major changes made to this specfile.


This runs fine on mock as well.

I was thinking of removing this request and filing another one for chardet2, but I was going to discuss this with the GitLab guys before doing that. Nevermind, though.
Comment 5 Vít Ondruch 2013-05-29 04:37:04 EDT
Anuj, as I see the upstream ticket, they are not really helpful unfortunately :/ They don't mind to propose even yet another fork, which is exactly the opposite of what we want. Could you please try to contact the original chardet author and try it the other way around? I.e. the original author will revive the development, or give them commit access and allow them to push new gems to rubygems.org. Thank you.
Comment 7 Vít Ondruch 2016-01-04 03:59:38 EST
Closing this stalled review.

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