Bug 628631 - Review Request: rubygem-httparty - Makes HTTP fun! Also, makes consuming restful web services dead easy
Summary: Review Request: rubygem-httparty - Makes HTTP fun! Also, makes consuming res...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-08-30 14:59 UTC by Michael Stahnke
Modified: 2010-09-11 16:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-09-11 16:11:14 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Michael Stahnke 2010-08-30 14:59:44 UTC
Spec URL: http://stahnma.fedorapeople.org/reviews/rubygem-httparty.spec
SRPM URL: http://stahnma.fedorapeople.org/reviews/rubygem-httparty-0.6.1-1.fc15.src.rpm
Description:  Makes HTTP fun! Also, makes consuming restful web services dead easy

Comment 1 Mamoru TASAKA 2010-09-05 21:01:07 UTC
Well, I will take this one and bug 630054.
Instead I will appreciate it if you would review my review requests
bug 630480 and bug 630481.

Note that I am going to take a rest now...

Comment 2 Mamoru TASAKA 2010-09-06 16:29:54 UTC
Some notes:

! BR(check)
  - From ./spec/spec_helper.rb :
--------------------------------------------------------
     1  require File.join(File.dirname(__FILE__), '..', 'lib', 'httparty')
     2  require 'spec/autorun'
     3  require 'fakeweb'
--------------------------------------------------------
    For BR(check): rubygem(fakeweb) seems also needed. I hope someone
    will package this.

* Requires
  - Usually ">= 0.1.8" on "R: rubygem(crack)" is not needed (is usually removed)
    because rubygem-crack on currently supported Fedora branches all satisfy
    this dependency, ref:
    https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires

    However, looking at gemspecfile, httparty-0.6.1.gemspec contains:
--------------------------------------------------------
    29      if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
    30        s.add_runtime_dependency(%q<crack>, ["= 0.1.8"])
--------------------------------------------------------
    So for this case, I suggest
    - to make the dependency for rubygem(crack) strict (i.e. use
      "R: rubygem(crack) = 0.1.8"
    - or remove strict version dependency for rubygem-crack on gemspec file

By the way, would you import rubygem-crack also for F-14/13?

Comment 3 Michael Stahnke 2010-09-07 16:30:31 UTC
I haven't put crack in F-14/F13 yet because shoulda isn't there yet.  I am working with the maintainer on that. 

Changed to a strict dependency for rubygem-crack.  = 0.1.8

New version

http://stahnma.fedorapeople.org/reviews/rubygem-httparty-0.6.1-2.fc15.src.rpm

Comment 4 Mamoru TASAKA 2010-09-07 19:37:18 UTC
Approving

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

Comment 5 Michael Stahnke 2010-09-07 20:11:10 UTC
New Package SCM Request
=======================
Package Name: rubygem-httparty
Short Description:  Makes HTTP fun! Also, makes consuming restful web services dead easy
Owners: stahnma
Branches: F13 F14 EL5 EL6 
InitialCC:

Comment 6 Kevin Fenzi 2010-09-08 17:58:37 UTC
Git done (by process-git-requests).

Comment 7 Mamoru TASAKA 2010-09-11 16:11:14 UTC
Closing.


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