Bug 727155
Summary: | Review Request: rubygem-fast_gettext - A simple, fast, memory-efficient and threadsafe implementation of GetText | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Vít Ondruch <vondruch> |
Component: | Package Review | Assignee: | Bohuslav "Slavek" Kabrda <bkabrda> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | bkabrda, notting, package-review |
Target Milestone: | --- | Flags: | bkabrda:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-10-20 12:59:49 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 705519, 738590 |
Description
Vít Ondruch
2011-08-01 12:58:17 UTC
* Licensing: The mentioned files in lib/vendor directory are not Ruby or GPLv2+, but Ruby or LGPLv2+ -- see https://github.com/mutoh/gettext/blob/master/README.rdoc (the library, where the files are from). * It would be good to query upstream not to bundle the gettext library, but use it as a separate dependency. * I would consider moving readme.md into the main package, as it contains information about licensing, which I think should be present in the main package. Otherwise, the package seems to be ok, but I would like to get the three points above clear. (In reply to comment #1) > * Licensing: The mentioned files in lib/vendor directory are not Ruby or > GPLv2+, but Ruby or LGPLv2+ -- see > https://github.com/mutoh/gettext/blob/master/README.rdoc (the library, where > the files are from). The file you are referring says: "This program is licenced under the same licence as Ruby", i.e. GPLv2 or Ruby => I am going to remove the "+", with the exception of 'setup.rb' file which is not present in the folder. So it is not LGPLv2+ IMO. > * It would be good to query upstream not to bundle the gettext library, but use > it as a separate dependency. I have checked the files once more. Although the files are coming from gettext library, it seems that they are more or less modified. Moreover, the original gettext package seems to be death, so it makes no sense IMO. > * I would consider moving readme.md into the main package, as it contains > information about licensing, which I think should be present in the main > package. DONE. > Otherwise, the package seems to be ok, but I would like to get the three points > above clear. Please note that I have also updated the package to the latest upstream version. Spec URL: http://people.redhat.com/vondruch/rubygem-fast_gettext.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-fast_gettext-0.6.1-1.fcf17.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3443784 (In reply to comment #2) > (In reply to comment #1) > > * Licensing: The mentioned files in lib/vendor directory are not Ruby or > > GPLv2+, but Ruby or LGPLv2+ -- see > > https://github.com/mutoh/gettext/blob/master/README.rdoc (the library, where > > the files are from). > > The file you are referring says: "This program is licenced under the same > licence as Ruby", i.e. GPLv2 or Ruby => I am going to remove the "+", with the > exception of 'setup.rb' file which is not present in the folder. So it is not > LGPLv2+ IMO. > I was refering to "... same licence as Ruby(See COPYING) or LGPL ...". So the whole license tag should be something like License: Public Domain and ((GPLv2+ or Ruby) or LGPLv2+) > > * It would be good to query upstream not to bundle the gettext library, but use > > it as a separate dependency. > > I have checked the files once more. Although the files are coming from gettext > library, it seems that they are more or less modified. Moreover, the original > gettext package seems to be death, so it makes no sense IMO. > I agree that the modifications made to files from gettext are non-trivial and that gettext is more or less dead. So it is not a problem for this package. > > * I would consider moving readme.md into the main package, as it contains > > information about licensing, which I think should be present in the main > > package. > > DONE. > > > Otherwise, the package seems to be ok, but I would like to get the three points > > above clear. > > > Please note that I have also updated the package to the latest upstream > version. > > Spec URL: http://people.redhat.com/vondruch/rubygem-fast_gettext.spec > SRPM URL: > http://people.redhat.com/vondruch/rubygem-fast_gettext-0.6.1-1.fcf17.src.rpm > > Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3443784 Before this package is approved, I would certainly like to make the License clear, otherwise everything is ok. (In reply to comment #3) > I was refering to "... same licence as Ruby(See COPYING) or LGPL ...". So the > whole license tag should be something like Please could you be more specific? The only reference to LGPL is in file "usr/lib/ruby/gems/1.8/gems/fast_gettext-0.6.1/lib/fast_gettext/vendor/README.rdoc" on line 155. This refers to file which is not included in the gem an will never be. Am I missing something? (In reply to comment #4) > (In reply to comment #3) > > I was refering to "... same licence as Ruby(See COPYING) or LGPL ...". So the > > whole license tag should be something like > > Please could you be more specific? The only reference to LGPL is in file > "usr/lib/ruby/gems/1.8/gems/fast_gettext-0.6.1/lib/fast_gettext/vendor/README.rdoc" > on line 155. This refers to file which is not included in the gem an will never > be. Am I missing something? Sorry, my mistake. I was looking at a newer license of the forked gettext library. The version that was forked was licensed under Ruby at the time, so your License field is OK. This package is APPROVED. Thank you for your review! New Package SCM Request ======================= Package Name: rubygem-fast_gettext Short Description: A simple, fast, memory-efficient and threadsafe implementation of GetText Owners: vondruch Branches: InitialCC: Git done (by process-git-requests). Package Change Request ====================== Package Name: rubygem-fast_gettext New Branches: f16 Owners: vondruch InitialCC: Git done (by process-git-requests). |