Bug 622904 - Review Request: rubygem-state_machine - Adds support for creating state machines for attributes on any Ruby class
Review Request: rubygem-state_machine - Adds support for creating state machi...
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-08-10 14:29 EDT by Guillermo Gómez
Modified: 2011-03-04 00:54 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-09-01 15:44:10 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mastahnke: fedora‑review+
a.badger: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Guillermo Gómez 2010-08-10 14:29:47 EDT
Spec URL: http://gomix.fedorapeople.org/rubygem-state_machine/rubygem-state_machine.spec
SRPM URL: http://gomix.fedorapeople.org/rubygem-state_machine/rubygem-state_machine-0.9.4-1.fc13.src.rpm
Description: Adds support for creating state machines for attributes on any
           : Ruby class

$ rpmlint -i SPECS/rubygem-state_machine.spec SRPMS/rubygem-state_machine-0.9.4-1.fc13.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.
Comment 1 Mamoru TASAKA 2010-08-10 14:37:01 EDT
rubygem-state_machine is already in Fedora.

Is it correct that this review request is for recheck of 
this, because it is long (actually almost one year) since
the last rubygem-state_machine is built on koji?
Comment 2 Guillermo Gómez 2010-08-10 15:16:58 EDT
I understand that it has to be treated as a new pkg since the last update its one year old.

quote

Re-review required for older packages
If a package was last updated more than three months ago (running cvs log -r HEAD *.spec can show you this information), you will need to submit a review request and have the package approved by a reviewer as if it were new to Fedora.

am i getting it wrong ? 

http://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers
Comment 3 Mamoru TASAKA 2010-08-10 15:44:33 EDT
Well, just a confirmation, thanks.
Comment 4 Michael Stahnke 2010-08-10 18:40:04 EDT
Macro %sitelib is not used anywhere.  It can be removed.

Package builds in mock without issues.

rpmlint lists normal warnings about macros.  

I have a small wish that there was a -doc package as 573 files of 737 are documentation, but it doesn't take up that much space.  

In some of my packages %{geminstdir}/test is marked as %doc and not in others.  I am guessing we need to agree on this one.  I'll ask on the list. 

This package looks like it is in pretty good shape.
Comment 5 Guillermo Gómez 2010-08-15 15:26:07 EDT
New spec and srpm, -doc pkg produced.

http://gomix.fedorapeople.org/rubygem-state_machine/rubygem-state_machine.spec

http://gomix.fedorapeople.org/rubygem-state_machine/rubygem-state_machine-0.9.4-2.fc13.src.rpm

$ rpmlint -i rubygem-state_machine.spec rubygem-state_machine-0.9.4-2.fc13.src.rpm
1 packages and 1 specfiles checked; 0 errors, 0 warnings.

wish granted :)
Comment 6 Guillermo Gómez 2010-08-20 10:14:56 EDT
Any news on this review? :)
Comment 7 Michael Stahnke 2010-08-20 10:41:57 EDT
rubygem-state_machine-doc.noarch: W: summary-ended-with-dot C Documentation files, rdoc, ri, examples and tests.

Just fix that when you import the package. 

Package APPROVED.
Comment 8 Guillermo Gómez 2010-08-20 11:53:25 EDT
Thank u Michael... :)

updates requests done for updates-testing with the fix

https://admin.fedoraproject.org/updates/search/rubygem-state_machine
Comment 9 Mamoru TASAKA 2010-09-01 13:47:20 EDT
What is the status of this bug? Can this bug be closed?
Comment 10 Guillermo Gómez 2010-09-01 15:44:10 EDT
Thanks Mamoru, closed ! Its on F12 and F13 stable already.
Comment 11 Guillermo Gómez 2011-02-03 15:42:32 EST
Package Change Request
======================
Package Name: rubygem-state_machine
New Branches: el5 el6
Owners: gomix
Comment 12 Guillermo Gómez 2011-02-08 16:37:24 EST
New Package SCM Request
=======================
Package Name: rubygem-state_machine
Short Description: Adds support for creating state machines for attributes on any Ruby class
Owners: gomix
Branches: el6 el5
InitialCC: gomix
Comment 13 Toshio Ernie Kuratomi 2011-02-11 13:36:14 EST
Git done.
Comment 14 Fedora Update System 2011-02-15 15:50:36 EST
rubygem-state_machine-0.9.4-6.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/rubygem-state_machine-0.9.4-6.el5
Comment 15 Fedora Update System 2011-02-15 16:13:29 EST
rubygem-state_machine-0.9.4-6.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/rubygem-state_machine-0.9.4-6.el6
Comment 16 Fedora Update System 2011-03-04 00:54:36 EST
rubygem-state_machine-0.9.4-6.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 17 Fedora Update System 2011-03-04 00:54:49 EST
rubygem-state_machine-0.9.4-6.el6 has been pushed to the Fedora EPEL 6 stable repository.  If problems still persist, please make note of it in this bug report.

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