Bug 711229
Summary: | Review Request: ruby-spqr - easy QMF agent framework for Ruby | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Will Benton <willb> |
Component: | Package Review | Assignee: | Tom "spot" Callaway <tcallawa> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, matt, notting, tcallawa, vondruch |
Target Milestone: | --- | Flags: | tcallawa:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | ruby-spqr-0.3.5-1.fc16 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2012-01-14 03:57:24 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: |
Description
Will Benton
2011-06-06 21:52:47 UTC
A new SRPM is available at http://packages.getwallaby.com/ruby-spqr-0.3.3-4.fc14.src.rpm The most recent SRPM is available from packages.getwallaby.com. == Review == Good: - rpmlint checks return: ruby-spqr.noarch: W: spurious-executable-perm /usr/share/doc/ruby-spqr-0.3.4/examples/hello.rb ruby-spqr.noarch: W: doc-file-dependency /usr/share/doc/ruby-spqr-0.3.4/examples/logservice.rb /usr/bin/env spqr-gen.noarch: W: spelling-error Summary(en_US) xml -> XML, ml, x ml spqr-gen.noarch: W: no-manual-page-for-binary spqr-gen.rb All safe to ignore. - package meets naming guidelines - package meets packaging guidelines - license (ASL 2.0) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream (d3ef30d1000f1f00187cc1a6d59d8713e37701b21d4ff4c90368f55a33ad6e5a) - package compiles on f16 (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file It all looks good, so I'm marking this as APPROVED. One minor item: I don't think there is a need for Requires: ruby, since you have Requires: ruby(abi) = 1.8, but this should be safe to remove before committing to Fedora git. Hi, could you please enlighten me why did you packaged the library as Ruby library instead of RubyGem? New Package SCM Request ======================= Package Name: ruby-spqr Short Description: easy QMF agent framework for Ruby Owners: willb Branches: f15 f16 InitialCC: Git done (by process-git-requests). ruby-spqr-0.3.5-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/ruby-spqr-0.3.5-1.fc16 ruby-spqr-0.3.5-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/ruby-spqr-0.3.5-1.fc15 ruby-spqr-0.3.5-1.fc15 has been pushed to the Fedora 15 testing repository. ruby-spqr-0.3.5-1.fc15 has been pushed to the Fedora 15 stable repository. ruby-spqr-0.3.5-1.fc15 has been pushed to the Fedora 15 stable repository. ruby-spqr-0.3.5-1.fc16 has been pushed to the Fedora 16 stable repository. |