Bug 988818 - Review Request: rubygem-xmlhash - A small C module that wraps libxml2's xmlreader to parse a XML string into a ruby hash
Summary: Review Request: rubygem-xmlhash - A small C module that wraps libxml2's xmlre...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Josef Stribny
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-26 13:27 UTC by Miroslav Suchý
Modified: 2016-01-04 05:51 UTC (History)
4 users (show)

Fixed In Version: rubygem-xmlhash-1.3.5-5.fc19
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-08-15 02:37:01 UTC
Type: ---
Embargoed:
jstribny: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Miroslav Suchý 2013-07-26 13:27:19 UTC
Spec URL: http://miroslav.suchy.cz/fedora/rubygem-xmlhash/rubygem-xmlhash.spec
SRPM URL: http://miroslav.suchy.cz/fedora/rubygem-xmlhash/rubygem-xmlhash-1.3.5-4.fc19.src.rpm
Description: A small C module that wraps libxml2's xmlreader to parse a XML
string into a ruby hash.

Fedora Account System Username: msuchy
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5660052

Comment 1 Josef Stribny 2013-08-05 12:07:50 UTC
Taking.

Comment 2 Josef Stribny 2013-08-05 12:47:46 UTC
* Use %%gem_dir to escape macro in comment
  rubygem-xmlhash.spec:50: W: macro-in-comment %gem_dir

* Adjust summary
  rubygem-xmlhash.src: E: summary-too-long C A small C module that wraps libxml2's xmlreader to parse a XML string into a ruby hash

* Move %doc %{gem_instdir}/README.txt to the main package since it contains licensing information, ideally you could ask upstream to provide a separate license file.

* I believe that /ext files used to build the extension could be probably put into -doc subpackage for clarification.

* Run the test suite with:
  testrb -Ilib test/test_xmlhash.rb so the library is properly loaded and tests run.

Other than that I have no complains. It builds, installs and works in mock. Please fix the issues above.

Comment 3 Miroslav Suchý 2013-08-05 13:18:58 UTC
> * I believe that /ext files used to build the extension could be probably put into -doc subpackage for clarification.

I would rather keep it just in .src.rpm

Other items updated:

Spec URL: http://miroslav.suchy.cz/fedora/rubygem-xmlhash/rubygem-xmlhash.spec
SRPM URL: http://miroslav.suchy.cz/fedora/rubygem-xmlhash/rubygem-xmlhash-1.3.5-5.fc19.src.rpm

Comment 4 Josef Stribny 2013-08-05 13:48:00 UTC
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5778895

APPROVING.

Comment 5 Miroslav Suchý 2013-08-05 13:58:48 UTC
New Package SCM Request
=======================
Package Name: rubygem-xmlhash
Short Description: A small C module to parse a XML string into a ruby hash
Owners: msuchy
Branches: f19
InitialCC:

Comment 6 Gwyn Ciesla 2013-08-05 14:28:22 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2013-08-05 15:16:32 UTC
rubygem-xmlhash-1.3.5-5.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/rubygem-xmlhash-1.3.5-5.fc19

Comment 8 Fedora Update System 2013-08-06 23:33:54 UTC
rubygem-xmlhash-1.3.5-5.fc19 has been pushed to the Fedora 19 testing repository.

Comment 9 Fedora Update System 2013-08-15 02:37:01 UTC
rubygem-xmlhash-1.3.5-5.fc19 has been pushed to the Fedora 19 stable repository.


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