Bug 1063039 - Review Request: rubygem-em-http-request - EventMachine based, async HTTP Request client
Summary: Review Request: rubygem-em-http-request - EventMachine based, async HTTP Requ...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mo Morsi
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2014-02-09 17:59 UTC by Nitesh Narayan Lal
Modified: 2016-02-08 13:56 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2015-07-22 06:42:19 UTC
Type: ---
mmorsi: fedora-review?

Attachments (Terms of Use)

Description Nitesh Narayan Lal 2014-02-09 17:59:35 UTC
Spec URL: http://niteshnarayan.fedorapeople.org/SPECS/rubygem-em-http-request-1.1.2.spec
SRPM URL: http://niteshnarayan.fedorapeople.org/SRPMS/rubygem-em-http-request-1.1.2-1.fc19.src.rpm 
Description: EventMachine based, async HTTP Request client
Fedora Account System Username:

Comment 1 Mo Morsi 2014-02-20 14:20:24 UTC
Taking this one. Couple of preliminary things:

- this depends on rubygem-http_parser.rb which isn't in Fedora, that will need to be built & pushed before this can be officially built in koji. Will wait till we sort out that gem before running the builds for this

- There seems to be an extraneous %gem_install in the %check section, please remove

- Please consider rm'ing the directories which you currently %exclude in the file list. Would result in a much cleaner / consolidated files list

- Upstream indicates the project is licensed under the MIT license in the README but there is no separate license file. Please file an issue w/ the upstream project to include this


- Note while it doesn't directly impact this gem, this upstream issue is related:


Comment 2 Miroslav Suchý 2015-07-21 15:01:39 UTC
Nitash, any progress here?

Comment 3 Nitesh Narayan Lal 2015-07-21 17:07:05 UTC
Nope unfortunately I am surrounded by loads of day job and other responsibilities. I didn't get time to continue this further, sorry for that.

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