Bug 443250
Summary: | Review Request: rubygem-rubyforge - A script which automates a limited set of rubyforge operations | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Darryl L. Pierce <dpierce> |
Component: | Package Review | Assignee: | Tom "spot" Callaway <tcallawa> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting, tross |
Target Milestone: | --- | Flags: | tcallawa:
fedora-review+
kevin: 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: | 2008-05-14 18:46:44 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
Darryl L. Pierce
2008-04-19 19:13:09 UTC
Review on -2: Good: - rpmlint checks return: rubygem-rubyforge.src: W: mixed-use-of-spaces-and-tabs (spaces: line 41, tab: line 7) rubygem-rubyforge.noarch: E: non-executable-script /usr/lib/ruby/gems/1.8/gems/rubyforge-0.4.4/lib/rubyforge.rb 0644 - package meets naming guidelines - package meets packaging guidelines (generic and Ruby) - license (GPLv2+ or Ruby) OK, text not in package, matches source - spec file legible, in am. english - source matches upstream (a955f92ebbb47953c3333c7f8428d3f168506cc4) - package builds on devel (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file Please fix the non-executable-script error in a new SRPM/SPEC and I will approve. You may also want to advise upstream to properly license their code, only one file seems to mention the license, and the text of the license is not included in the Gem. This is not a blocker for Fedora inclusion. Updated the spec to set the execute bit on the rubyforge.rb script. Spec: http://mcpierce.dyndns.org/~mcpierce/rubygem-rubyforge.spec SRPM: http://mcpierce.dyndns.org/~mcpierce/rubygem-rubyforge-0.4.4-3.fc8.src.rpm APPROVED. New Package CVS Request ======================= Package Name: rubygem-rubyforge Short Description: A script which automates a limited set of rubyforge operations Owners: mcpierce Branches: F-8 F-9 InitialCC: none Cvsextras Commits: Yes cvs done. This is now properly building against rawhide, F8 and F9. |