Bug 653407
Summary: | Review Request: rubygem-hashery - A collection of Hash-like classes for Ruby | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Marek Goldmann <mgoldman> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, mtasaka, notting |
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
j: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | rubygem-hashery-1.3.0-3.fc13 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-11-21 19:50:56 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: | 652396 |
Description
Marek Goldmann
2010-11-15 11:07:55 UTC
Some notes: * License ----------------------------------------------------- Overall: ASL 2.0 GPL+ ./lib/hashery/ini.rb ./lib/hashery/linkedlist.rb GPLv2 or Ruby ./lib/hashery/ostructable.rb ----------------------------------------------------- - The license tag should be "ASL 2.0 and GPL+ and (GPLv2 or Ruby)" ! Usability - It seems that hashery 1.3.0 gem is a bit broken ----------------------------------------------------- [tasaka1@localhost ~]$ irb irb(main):001:0> require "rubygems" ge=> true irb(main):002:0> gem "hashery" => true irb(main):003:0> require "hashery" LoadError: no such file to load -- facets/basicobject from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `gem_original_require' from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' from /usr/lib/ruby/gems/1.8/gems/hashery-1.3.0/lib/hashery/openobject.rb:1 from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `gem_original_require' from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' from /usr/lib/ruby/gems/1.8/gems/hashery-1.3.0/lib/hashery.rb:7 from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `gem_original_require' from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' from (irb):3 ----------------------------------------------------- Currently I won't make this a blocker for this review request ( as requiring scripts under lib/hashery/ directly seems to work,] except for "lib/hashery/openobject.rb"), However would you contact the upstream? ? Scripts under test/ - How can scripts under test/ directory be used? (If some test programs can be executable, would you do some tests in %check?) (In reply to comment #1) > Some notes: > > * License > ----------------------------------------------------- > Overall: ASL 2.0 > > GPL+ > ./lib/hashery/ini.rb > ./lib/hashery/linkedlist.rb > > GPLv2 or Ruby > ./lib/hashery/ostructable.rb > ----------------------------------------------------- > - The license tag should be "ASL 2.0 and GPL+ and (GPLv2 or Ruby)" Fixed! > ! Usability > - It seems that hashery 1.3.0 gem is a bit broken > ----------------------------------------------------- > [tasaka1@localhost ~]$ irb > irb(main):001:0> require "rubygems" > ge=> true > irb(main):002:0> gem "hashery" > => true > irb(main):003:0> require "hashery" > LoadError: no such file to load -- facets/basicobject > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in > `gem_original_require' > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' > from /usr/lib/ruby/gems/1.8/gems/hashery-1.3.0/lib/hashery/openobject.rb:1 > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in > `gem_original_require' > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' > from /usr/lib/ruby/gems/1.8/gems/hashery-1.3.0/lib/hashery.rb:7 > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in > `gem_original_require' > from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `require' > from (irb):3 > ----------------------------------------------------- > Currently I won't make this a blocker for this review request > ( as requiring scripts under lib/hashery/ directly seems to work,] > except for "lib/hashery/openobject.rb"), However would you contact the > upstream? Sure, created new issue for that: https://github.com/rubyworks/hashery/issues/issue/5 > ? Scripts under test/ > - How can scripts under test/ directory be used? (If some test programs can > be executable, would you do some tests in %check?) Well, it's a bit complicated. Tests are written using Lemon framework (https://github.com/proutils/lemon https://rubygems.org/gems/lemon). This framework has big dependency tree which is not packaged as RPMs. Is this a hard require to execute tests? Could we add it in next versions when the tree will be at least half-packaged? Updated files: Spec URL: http://goldmann.fedorapeople.org/package_review/rubygem-hashery.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/rubygem-hashery-1.3.0-2.fc14.src.rpm Thanks! (In reply to comment #2) > (In reply to comment #1) > > > ? Scripts under test/ > > - How can scripts under test/ directory be used? (If some test programs can > > be executable, would you do some tests in %check?) > > Well, it's a bit complicated. Tests are written using Lemon framework > (https://github.com/proutils/lemon https://rubygems.org/gems/lemon). This > framework has big dependency tree which is not packaged as RPMs. Is this a hard > require to execute tests? Could we add it in next versions when the tree will > be at least half-packaged? - Well, actually it looks somewhat complicated, so I would make %check section a blocker for this package (if some simple test method gets available (in the future), please consider to add it in %check) ---------------------------------------------------------- This package (rubygem-hashery) is APPROVED by mtasaka ---------------------------------------------------------- Thanks for review! New Package SCM Request ======================= Package Name: rubygem-hashery Short Description: A collection of Hash-like classes for Ruby Owners: goldmann Branches: f13 f14 Git done (by process-git-requests). rubygem-hashery-1.3.0-2.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-hashery-1.3.0-2.fc13 rubygem-hashery-1.3.0-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-hashery-1.3.0-2.fc14 rubygem-hashery-1.3.0-2.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-hashery'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-hashery-1.3.0-2.fc14 Please rebuild this package also for rawhide (F-15). Thank you. Now closing. rubygem-hashery-1.3.0-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-hashery-1.3.0-3.fc14 rubygem-hashery-1.3.0-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-hashery-1.3.0-3.fc13 Updated package after reporting "require 'hashery'" issues with upstream: https://github.com/rubyworks/hashery/issues/5 rubygem-hashery-1.3.0-3.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. rubygem-hashery-1.3.0-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. |