[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #55 from Fedora Update System upda...@fedoraproject.org 
2011-05-24 22:31:04 EDT ---
rubygem-heroku-2.0.4-1.fc13 has been pushed to the Fedora 13 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||rubygem-heroku-2.0.4-1.fc13
 Resolution||ERRATA
Last Closed||2011-05-24 22:31:10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #56 from Fedora Update System upda...@fedoraproject.org 
2011-05-24 23:01:54 EDT ---
rubygem-heroku-2.0.4-1.fc14 has been pushed to the Fedora 14 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|rubygem-heroku-2.0.4-1.fc13 |rubygem-heroku-2.0.4-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|rubygem-heroku-2.0.4-1.fc14 |rubygem-heroku-2.0.4-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #57 from Fedora Update System upda...@fedoraproject.org 
2011-05-24 23:24:42 EDT ---
rubygem-heroku-2.0.4-1.fc15 has been pushed to the Fedora 15 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #53 from Fedora Update System upda...@fedoraproject.org 
2011-05-01 15:37:40 EDT ---
rubygem-heroku-2.0.4-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-heroku-2.0.4-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #52 from Fedora Update System upda...@fedoraproject.org 
2011-05-01 15:37:30 EDT ---
rubygem-heroku-2.0.4-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-2.0.4-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-05-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #54 from Fedora Update System upda...@fedoraproject.org 
2011-05-01 15:37:48 EDT ---
rubygem-heroku-2.0.4-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-heroku-2.0.4-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-04-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #49 from Fedora Update System upda...@fedoraproject.org 
2011-04-23 01:22:41 EDT ---
rubygem-heroku-1.20.1-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.20.1-2.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-04-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #50 from Fedora Update System upda...@fedoraproject.org 
2011-04-23 01:22:51 EDT ---
rubygem-heroku-1.20.1-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.20.1-2.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-04-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #51 from Fedora Update System upda...@fedoraproject.org 
2011-04-23 01:22:59 EDT ---
rubygem-heroku-1.20.1-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.20.1-2.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-04-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #48 from Fedora Update System upda...@fedoraproject.org 
2011-04-10 03:39:49 EDT ---
rubygem-heroku-1.20.1-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.20.1-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #47 from Fedora Update System upda...@fedoraproject.org 
2011-01-13 11:55:53 EST ---
rubygem-heroku-1.17.5-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.17.5-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #46 from Fedora Update System upda...@fedoraproject.org 
2011-01-13 11:55:46 EST ---
rubygem-heroku-1.17.5-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.17.5-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #44 from Fedora Update System upda...@fedoraproject.org 
2011-01-08 15:46:54 EST ---
rubygem-heroku-1.16.2-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.16.2-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #45 from Fedora Update System upda...@fedoraproject.org 
2011-01-08 15:47:01 EST ---
rubygem-heroku-1.16.2-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.16.2-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #43 from Fedora Update System upda...@fedoraproject.org 
2011-01-04 15:58:46 EST ---
rubygem-heroku-1.15.1-1.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-heroku'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/rubygem-heroku-1.15.1-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #42 from Fedora Update System upda...@fedoraproject.org 
2011-01-03 16:28:20 EST ---
rubygem-heroku-1.15.1-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.15.1-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #41 from Fedora Update System upda...@fedoraproject.org 
2011-01-03 16:28:13 EST ---
rubygem-heroku-1.15.1-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-heroku-1.15.1-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #40 from Kevin Fenzi ke...@tummy.com 2011-01-02 15:14:17 EST ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2011-01-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #39 from Minnikhanov minnikha...@gmail.com 2011-01-01 08:53:36 
EST ---
(In reply to comment #35)
 (In reply to comment #33)
  heroku.gem have spec/ folder.
  
  Is need to realize %check section?
  Like that
  #%check
  #pushd %{buildroot}/%{geminstdir}
  #%_bindir/spec spec
  #popd
 
 - Yes, it is preferable, however for this gem $ spec spec/
   needs webmock gem, which is not in Fedora and currently review
   request for rubygem-webmock does not exist either (once
   existed but it was closed unfinished: bug 588477)
 
   If you use rubygem-webmock you can package the srpm for Fedora.

webmock gem have Runtime Dependencies addressable = 2.2.2
addressable.gem is not in Fedora, all it's Runtime Dependencies there are in
Fedora. 'httpclient.gem' is development dependency for addressable.gem

Is it possible to offer Adam Young (bug 588477) return to pack rubygem-webmock
and to pack rubygem-addressable also?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #37 from Minnikhanov minnikha...@gmail.com 2010-12-31 14:20:20 
EST ---
Thank you for the review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Minnikhanov minnikha...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #38 from Minnikhanov minnikha...@gmail.com 2010-12-31 14:25:37 
EST ---
New Package SCM Request
===
Package Name: rubygem-heroku
Short Description: Deploy apps to Heroku
Owners: minn
Branches: f13 f14
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #36 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-27 
12:30:04 EST ---
Okay.

-
This package (rubygem-heroku) is APPROVED by
mtasaka
-

Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from Add Package to Source Code Management (SCM) system and Set Owner

Now I am sponsoring you.

If you want to import this package into Fedora 13/14, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on koji Fedora rebuilding system).

When using Fedora SCM system, please check below for reference:
http://fedoraproject.org/wiki/Using_Fedora_GIT

If you have questions, please ask me.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #34 from Minnikhanov minnikha...@gmail.com 2010-12-24 13:17:13 
EST ---
heroku-1.15.1 22/12/2010

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.15.1-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #32 from Minnikhanov minnikha...@gmail.com 2010-12-24 09:28:19 
EST ---
heroku.gem have Runtime Dependencies = launchy ~ 0.3.2

Now my template generate
Requires: rubygem(launchy) = 0.3.2

 Is need to generate 2 Requires?
Sample:
Requires: rubygem(launchy) = 0.3.2
Requires: rubygem(launchy)  0.4

Common algorithm:
versions = a.b.c.d
versions =~ /\d+(\.\d*)*/

1. Runtime Dependencies = GEM ~ a.b.c.d
generate:
Requires: rubygem(GEM) = a.b.c.d
Requires: rubygem(GEM)  a.b.c+1

2. Runtime Dependencies = GEM ~ a.b.c.
generate:
Requires: rubygem(GEM) = a.b.c   - is need here last dot (a.b.c.)
Requires: rubygem(GEM)  a.b.c+1

3. Runtime Dependencies = GEM ~ a.
   Runtime Dependencies = GEM ~ a
generate:
Requires: rubygem(GEM) = a   - is need here last dot (a.)
Requires: rubygem(GEM)  a+1

4. Runtime Dependencies = GEM ~
generate:
Requires: rubygem(GEM) = 0
Requires: rubygem(GEM)  1

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #33 from Minnikhanov minnikha...@gmail.com 2010-12-24 09:38:00 
EST ---
heroku.gem have spec/ folder.

Is need to realize %check section?
Like that
#%check
#pushd %{buildroot}/%{geminstdir}
#%_bindir/spec spec
#popd

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #35 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-25 
11:44:42 EST ---
Well,

(In reply to comment #32)
 heroku.gem have Runtime Dependencies = launchy ~ 0.3.2
 
 Now my template generate
 Requires: rubygem(launchy) = 0.3.2
 
  Is need to generate 2 Requires?
 Sample:
 Requires: rubygem(launchy) = 0.3.2
 Requires: rubygem(launchy)  0.4

- Usually we write Requires: rubygem(launchy) = 0.3.2 part
  only, however technically writing also 
  Requires: rubygem(launchy)  0.4 is not wrong (according to
  rubygem's dependency format).

  I asked on fedora-packaging mailing list how people think about
  writing both Requires (especially for writing lower Requires):
  http://lists.fedoraproject.org/pipermail/packaging/2010-November/007501.html

  ... and got no answer, so for now I think that you can write
  Requires: rubygem(launchy)  0.4 also _if you want_. 


(In reply to comment #33)
 heroku.gem have spec/ folder.
 
 Is need to realize %check section?
 Like that
 #%check
 #pushd %{buildroot}/%{geminstdir}
 #%_bindir/spec spec
 #popd

- Yes, it is preferable, however for this gem $ spec spec/
  needs webmock gem, which is not in Fedora and currently review
  request for rubygem-webmock does not exist either (once
  existed but it was closed unfinished: bug 588477)

  If you use rubygem-webmock you can package the srpm for Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #31 from Minnikhanov minnikha...@gmail.com 2010-12-23 15:14:39 
EST ---
(In reply to comment #29)
 For rubygem-heroku side, one comment
 
 - Please put one line between each %changelog entry (because
   it is useful when using Fedora git system) like:
 ---
 * Tue Dec 21 2010  minnikha...@gmail.com - 1.15.0-1
 - Initial package
 
 * Thu Dec 21 2010  minnikha...@gmail.com - 1.14.10-3
 - Fix Comment 23 #661436 (Review Request)
 - Remove '/.yardoc'.
 
 * Thu Dec 18 2010  minnikha...@gmail.com - 1.14.10-2
 - Fix Comment 18 #661436 (Review Request)
 - Set Release: 2
 ---
Fix it.

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.15.0-2.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #29 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-22 
14:47:37 EST ---
For rubygem-heroku side, one comment

- Please put one line between each %changelog entry (because
  it is useful when using Fedora git system) like:
---
* Tue Dec 21 2010  minnikha...@gmail.com - 1.15.0-1
- Initial package

* Thu Dec 21 2010  minnikha...@gmail.com - 1.14.10-3
- Fix Comment 23 #661436 (Review Request)
- Remove '/.yardoc'.

* Thu Dec 18 2010  minnikha...@gmail.com - 1.14.10-2
- Fix Comment 18 #661436 (Review Request)
- Set Release: 2
---

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #30 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-22 
14:49:18 EST ---
-
NOTE: Before being sponsored:

This package will be accepted with another few (or no) work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to show that you have an understanding 
of the process and of the packaging guidelines as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a pre-review of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on my wiki page:
http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets
(Check No one is reviewing)

Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets



!! If you are interested in ruby related things, packaging one
   more rubygem based packages makes sponsoring process faster.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #26 from Minnikhanov minnikha...@gmail.com 2010-12-21 14:21:31 
EST ---
(In reply to comment #23)
  
  Does not build...
  http://koji.fedoraproject.org/koji/taskinfo?taskID=2675006
  
 
 I see build.log of this task - build failed.

Fix it.

http://koji.fedoraproject.org/koji/taskinfo?taskID=2681968 
koji scratch build successful.

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.10-3.fc14.src.rpm

Just will make heroku-1.15.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #27 from Minnikhanov minnikha...@gmail.com 2010-12-21 14:53:17 
EST ---
(In reply to comment #23)
  
  Does not build...
  http://koji.fedoraproject.org/koji/taskinfo?taskID=2675006
  
 
 I see build.log of this task - build failed.

Fix it.

http://koji.fedoraproject.org/koji/taskinfo?taskID=2681968 
koji scratch build successful.

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.10-3.fc14.src.rpm

Just will make heroku-1.15.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #25 from Minnikhanov minnikha...@gmail.com 2010-12-20 15:22:24 
EST ---
(In reply to comment #23)
  
  Does not build...
  http://koji.fedoraproject.org/koji/taskinfo?taskID=2675006
  
 
 I see build.log of this task - build failed.
 I place /.yardoc into -doc subpackage - I was not right :-(
 I improve that, shift '/.yardoc' into main package and create next release
 (rubygem-heroku-1.14.10-3.fc14.src.rpm)
 

http://koji.fedoraproject.org/koji/taskinfo?taskID=2678768 - this I build from
my old 1st spec, without doc-subpackage. No error.

I try to check 'rpmbuild bb ...' by interupted (Ctrl-C) - '/.yardoc'
disappeared.
When I exclude '/.yardoc' from %files, 'rpmbuild bb ...' interupted by error.
Something wrong on define doc-subpackage.

'rpmbuild bb ...' clear all subfolder ~/rpmbuild. I don't find options for
prevent this clear. Do you know about that?

Will you route me to right way.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #24 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-19 
13:10:11 EST ---
(In reply to comment #23)

 Must I check myself by 'mock'? 
 Before place this Review Request I started to run, at one time, 
 'mock -r fedora-14-x86_64 rebuild ../SRPMS/rubygem-heroku*.rpm'
  after successful
 rpmlint rubygem-heroku.spec ../RPMS/*/rubygem-heroku*.rpm
 ../SRPMS/rubygem-heroku*.rpm

- Checking if your srpm builds with mock is preferable,
  and once you gets sponsored you can use koji scratch build
  instead. And anyway I check srpms in review requests
  with koji scratch build by myself so that I can tell the submitter
  the result of build test.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #14 from Minnikhanov minnikha...@gmail.com 2010-12-17 14:13:21 
EST ---
(In reply to comment #13)
 For 1.14.10-1:
 
 * Duplicate entries
   - build.log shows:
 -
 62  warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10
 63  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/bin
 64  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/bin/heroku
 65  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib
 66  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku
 67  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku-postgresql
 68  warning: File listed twice:
 /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku-postgresql/client.rb
 ...
 ...
 -
 Note that %files entry
 -
 %files
 %{geminstdir}/
 -
 contains the directory %geminstdir itself and all files/directories/etc
 under this directory, and this entry is now no longer needed.
 
 * Documents
   - Please move README.md to main package (from -doc subpackage) because
 this file contains license information.
 
 
 By the way are you already sponsored? If not, do you have
 another review request submission?

Fix all notes.

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.10-1.fc14.src.rpm

At github (don't download by 'wget')
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku-1.14.10-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #16 from Minnikhanov minnikha...@gmail.com 2010-12-17 14:44:42 
EST ---
(In reply to comment #13)

 By the way are you already sponsored? If not, do you have
 another review request submission?

I am not sponsored, I hope that will be. 
Next I'll plan to pack 
devise-1.0.9_20101209.gem - it is for rails ~2.3  3.
For run-time dependencies need warden.gem 
I want to use devise.gem in my rails app.

When I'll begin to build apps by hobo.gem, I'll hope to pack it.

Now I don't know what I must do else with rubygem-heroku. Is it enough for me
in packaging heroku - make spec  src.rpm files?
Begin devise?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #17 from Minnikhanov minnikha...@gmail.com 2010-12-17 14:54:14 
EST ---
(In reply to comment #15)
 Please change release number in the spec file (if version does not
 change) even if during review request to avoid confusion.

Fix it. URLs - Comment 14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #18 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-17 
15:10:46 EST ---
srpms in both comment 12 and comment 14 have same EVR 1.14.10-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #19 from Minnikhanov minnikha...@gmail.com 2010-12-18 01:52:20 
EST ---
(In reply to comment #18)
 srpms in both comment 12 and comment 14 have same EVR 1.14.10-1.fc14

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.10-2.fc14.src.rpm

At github (don't download by 'wget')
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku-1.14.10-2.fc14.src.rpm

Now I understand. 
Start point 'rubygem-heroku-1.14.10-2'.
Any change by Comment - Release : =+ 1

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #12 from Minnikhanov minnikha...@gmail.com 2010-12-16 14:08:19 
EST ---
heroku-1.14.10 16/12/2010

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.10-1.fc14.src.rpm

At github (don't download by 'wget')
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku-1.14.10-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?

--- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-16 
14:59:41 EST ---
For 1.14.10-1:

* Duplicate entries
  - build.log shows:
-
62  warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/heroku-1.14.10
63  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/bin
64  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/bin/heroku
65  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib
66  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku
67  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku-postgresql
68  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/heroku-1.14.10/lib/heroku-postgresql/client.rb
...
...
-
Note that %files entry
-
%files
%{geminstdir}/
-
contains the directory %geminstdir itself and all files/directories/etc
under this directory, and this entry is now no longer needed.

* Documents
  - Please move README.md to main package (from -doc subpackage) because
this file contains license information.


By the way are you already sponsored? If not, do you have
another review request submission?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #11 from Minnikhanov minnikha...@gmail.com 2010-12-15 14:45:08 
EST ---
Pack heroku-1.14.9 13/12/2010

Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.9-1.fc14.src.rpm

At github (don't download by 'wget')
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku-1.14.9-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-13 
12:31:51 EST ---
So would you update your spec / srpm?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #9 from Minnikhanov minnikha...@gmail.com 2010-12-13 13:31:33 EST 
---
(In reply to comment #8)
 So would you update your spec / srpm?

Yes, at both points - dropbox  github.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #10 from Minnikhanov minnikha...@gmail.com 2010-12-13 13:34:37 
EST ---
(In reply to comment #4)
 Some notes:
 
 * Unused macros
   - The defined macro %ruby_sitelib is used nowhere.
   - Please use the defined %geminstdir macro also in %files
 
 * License
   - The license is MIT
 
 * source URL
   - Please specify the full URL for Source0
 https://fedoraproject.org/wiki/Packaging/SourceURL
 
 * BuildRoot
   - On Fedora BuildRoot line is no longer needed:
 https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
 
 * documents / -doc subpackage
   - Please consider to split document files (which are not
 needed on runtime) to -doc subpackage.
 The following files/directories can be moved to -doc subpackage
 --
 %{gemdir}/doc/%{gemname}-%{version}/
 %{geminstdir}/spec/
 --
   - Please mark %geminstdir/README.md as %doc correctly.

Fix all notes.
Check doc-subpackage thoroughly - this my 1st subpackage. :-)

rpmlint - no error.
[...@lhost SPECS]$ rpmlint rubygem-heroku.spec
rubygem-heroku.spec: W: no-cleaning-of-buildroot %clean
rubygem-heroku.spec: W: no-buildroot-tag
rubygem-heroku.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 3 warnings.
[...@lhost SPECS]$

rpmbuild  -ba rubygem-heroku.spec  - no error.

Wrote: /home/pkg/rpmbuild/SRPMS/rubygem-heroku-1.14.8-1.fc14.src.rpm
Wrote: /home/pkg/rpmbuild/RPMS/noarch/rubygem-heroku-1.14.8-1.fc14.noarch.rpm
Wrote:
/home/pkg/rpmbuild/RPMS/noarch/rubygem-heroku-doc-1.14.8-1.fc14.noarch.rpm
Executing(%clean): /bin/sh -e /var/tmp/rpm-tmp.PF0kb5
+ umask 022
+ cd /home/pkg/rpmbuild/BUILD
+ /bin/rm -rf /home/pkg/rpmbuild/BUILDROOT/rubygem-heroku-1.14.8-1.fc14.x86_64
+ exit 0
[...@lhost SPECS]$

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #5 from Minnikhanov minnikha...@gmail.com 2010-12-12 07:26:12 EST 
---
(In reply to comment #0)
 
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-launchy.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-launchy-0.3.7-1.fc14.src.rpm
 Description: 
 Hi. I just finished packaging up rubygem-heroku and rubygem-launchy, and I
 would appreciate a review so that I can get it into Fedora Extras. 
 
 Heroku API - deploy apps to Heroku (http://heroku.com/) from the command line.
 launchy - Runtime Dependencies for heroku.gem


rubygem-launchy is already packaged for Fedora and also for EPEL.
Excuse me, I miss link 
http://koji.fedoraproject.org/koji/packageinfo?packageID=9756 .

I exclude rubygem-launchy from my github repo also.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #7 from Minnikhanov minnikha...@gmail.com 2010-12-12 09:02:50 EST 
---
(In reply to comment #0)
 Spec URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku.spec
 SRPM URL:
 https://github.com/minn/rubygem/blob/master/heroku/rubygem-heroku-1.14.8-1.fc14.src.rpm
 
Links above follows to download page of full github's repo.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #6 from Minnikhanov minnikha...@gmail.com 2010-12-12 08:59:49 EST 
---
(In reply to comment #1)

 Also, quickly checking the src.rpm, with rpm -qpl I get
 
 error: rubygem-heroku-1.14.8-1.fc14.src.rpm: not an rpm package (or package
 manifest)
 
 
 Perhaps you might want to check this url?
 

Thise links can get by 'wget':
Spec URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-heroku-1.14.8-1.fc14.src.rpm

Links above follows to download page of full github's repo.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

David Riches davi...@ultracar.co.uk changed:

   What|Removed |Added

 CC||davi...@ultracar.co.uk

--- Comment #1 from David Riches davi...@ultracar.co.uk 2010-12-10 11:14:04 
EST ---
*not a sponsor*, just some quick feedback

These packages really need to be separate bugzillas.

License: GPLv2+ or Ruby - I think you should probably pick one

Also, quickly checking the src.rpm, with rpm -qpl I get

error: rubygem-heroku-1.14.8-1.fc14.src.rpm: not an rpm package (or package
manifest)


Perhaps you might want to check this url?


Regards

Dave

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #2 from Minnikhanov minnikha...@gmail.com 2010-12-10 11:43:06 EST 
---
(In reply to comment #1)
 
 Also, quickly checking the src.rpm, with rpm -qpl I get
 
 error: rubygem-heroku-1.14.8-1.fc14.src.rpm: not an rpm package (or package
 manifest)
 
 
 Perhaps you might want to check this url?
 

I check github's url.
Now I downloaded my srpm from github, check it by rpm -qpl ... - no error. 
http://fpaste.org/YSwu/ 

[...@lhost heroku]$ ls
rubygem-heroku-1.14.6-1.fc14.src.rpm  rubygem-launchy-0.3.7-1.fc14.src.rpm
rubygem-heroku-1.14.8-1.fc14.src.rpm  rubygem-launchy.spec
rubygem-heroku.spec
[...@lhost heroku]$ rpm -qpl rubygem-heroku-1.14.8-1.fc14.src.rpm
heroku-1.14.8.gem
rubygem-heroku.spec
[...@lhost heroku]$ rpm -qpl rubygem-launchy-0.3.7-1.fc14.src.rpm
launchy-0.3.7.gem
rubygem-launchy.spec
[...@lhost heroku]$ ^C

Also check by rpmlint - no error.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

--- Comment #3 from Minnikhanov minnikha...@gmail.com 2010-12-10 11:50:37 EST 
---
(In reply to comment #1)
 
 License: GPLv2+ or Ruby - I think you should probably pick one
 

This generated by gem2rpm - I don't know which to pick.
Will anybody advice - what License is right.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 661436] Review Request: rubygem-heroku - deploy apps to Heroku

2010-12-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=661436

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp

--- Comment #4 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-12-10 
12:48:52 EST ---
Some notes:

* Unused macros
  - The defined macro %ruby_sitelib is used nowhere.
  - Please use the defined %geminstdir macro also in %files

* License
  - The license is MIT

* source URL
  - Please specify the full URL for Source0
https://fedoraproject.org/wiki/Packaging/SourceURL

* BuildRoot
  - On Fedora BuildRoot line is no longer needed:
https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

* documents / -doc subpackage
  - Please consider to split document files (which are not
needed on runtime) to -doc subpackage.
The following files/directories can be moved to -doc subpackage
--
%{gemdir}/doc/%{gemname}-%{version}/
%{geminstdir}/spec/
--
  - Please mark %geminstdir/README.md as %doc correctly.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review