Bug 1142491 - Review Request: rubygem-ox - Fast XML parser and object serializer
Summary: Review Request: rubygem-ox - Fast XML parser and object serializer
Keywords:
Status: CLOSED ERRATA
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: 2014-09-16 21:39 UTC by František Dvořák
Modified: 2015-04-22 18:46 UTC (History)
1 user (show)

Fixed In Version: rubygem-ox-2.1.8-2.fc22
Clone Of:
Environment:
Last Closed: 2015-04-21 19:16:28 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description František Dvořák 2014-09-16 21:39:12 UTC
Spec URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.3-1/rubygem-ox.spec
SRPM URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.3-1/rubygem-ox-2.1.3-1.fc22.src.rpm
Description: A fast XML parser and object serializer that uses only standard C lib. Optimized XML (Ox), as the name implies was written to provide speed optimized XML handling. It was designed to be an alternative to Nokogiri and other Ruby XML parsers for generic XML parsing and as an alternative to Marshal for Object serialization.
Fedora Account System Username: valtri

koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=7596555

Comment 1 František Dvořák 2014-12-31 22:37:12 UTC
Spec URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.6-1/rubygem-ox.spec
SRPM URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.6-1/rubygem-ox-2.1.6-1.fc22.src.rpm

koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=8508423

%changelog
* Wed Dec 31 2014 František Dvořák <valtri.cz> - 2.1.6-1
- Update to 2.1.6
- Changed license from BSD to MIT (https://github.com/ohler55/ox/issues/104)
- Tests added

Comment 2 František Dvořák 2015-02-10 16:10:05 UTC
Spec URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.7-1/rubygem-ox.spec
SRPM URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.7-1/rubygem-ox-2.1.7-1.fc22.src.rpm

koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=8884773

%changelog
* Tue Feb 10 2015 František Dvořák <valtri.cz> - 2.1.7-1
- Update to 2.1.7

Comment 3 František Dvořák 2015-02-17 13:57:48 UTC
Spec URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.8-1/rubygem-ox.spec
SRPM URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.8-1/rubygem-ox-2.1.8-1.fc23.src.rpm

koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=8966608

%changelog
* Tue Feb 17 2015 František Dvořák <valtri.cz> - 2.1.8-1
- Update to 2.1.8

Comment 4 Mamoru TASAKA 2015-03-20 07:17:34 UTC
Taking.

I would appreciate it if you would review my review
request (bug 1204009)

Comment 5 Mamoru TASAKA 2015-03-20 10:01:25 UTC
Some remarks:

* %license
  - Use %license for LICENSE file

* Document files placement
  - I would recommend to move README.md to main package
    because it says "README" (not a blocker)

* License
  - And while LICENSE file is MIT, actually lib/ox.rb seems
    under BSD. Would you check this?

Comment 6 František Dvořák 2015-03-23 16:02:44 UTC
(In reply to Mamoru TASAKA from comment #5)
> Some remarks:
> 
> * %license
>   - Use %license for LICENSE file
> 

Fixed.

> * Document files placement
>   - I would recommend to move README.md to main package
>     because it says "README" (not a blocker)
> 

OK. Some packagers place the README in the doc subpackage, but the main package may be slightly better. :-)

> * License
>   - And while LICENSE file is MIT, actually lib/ox.rb seems
>     under BSD. Would you check this?

Fixed upstream (there were change of the license to MIT recently):

https://github.com/ohler55/ox/pull/108


New version:

Spec URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.8-2/rubygem-ox.spec
SRPM URL: http://scientific.zcu.cz/fedora/rubygem-ox-2.1.8-2/rubygem-ox-2.1.8-2.fc23.src.rpm

%changelog
* Mon Mar 23 2015 František Dvořák <valtri.cz> - 2.1.8-2
- Use the %%license tag
- Move README.md to main package

Thanks for the review!

Comment 7 Mamoru TASAKA 2015-03-26 09:38:22 UTC
Okay, I would recommend to add some comments on the spec file for license clarification until the next version.

* spec file clean
* source contents no problem
* builds on F-23/22/21
* can be installed
* at least the following works

$ ruby -e "require 'ox' ; p Ox::VERSION"
"2.1.8"

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

Comment 8 František Dvořák 2015-03-26 18:32:09 UTC
New Package SCM Request
=======================
Package Name: rubygem-ox
Short Description: Fast XML parser and object serializer
Upstream URL: http://www.ohler.com/ox
Owners: valtri
Branches: f20 f21 f22 epel7
InitialCC:

Comment 9 Gwyn Ciesla 2015-03-27 12:33:32 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2015-03-27 14:02:22 UTC
rubygem-ox-2.1.8-2.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/rubygem-ox-2.1.8-2.fc22

Comment 11 Fedora Update System 2015-03-27 14:03:07 UTC
rubygem-ox-2.1.8-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/rubygem-ox-2.1.8-2.fc21

Comment 12 Fedora Update System 2015-03-27 14:03:47 UTC
rubygem-ox-2.1.8-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/rubygem-ox-2.1.8-2.fc20

Comment 13 Fedora Update System 2015-03-27 14:04:27 UTC
rubygem-ox-2.1.8-2.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/rubygem-ox-2.1.8-2.el7

Comment 14 Fedora Update System 2015-03-29 04:24:25 UTC
rubygem-ox-2.1.8-2.fc22 has been pushed to the Fedora 22 testing repository.

Comment 15 Fedora Update System 2015-04-08 06:56:37 UTC
rubygem-ox-2.1.8-2.fc21 has been pushed to the Fedora 21 stable repository.

Comment 16 Fedora Update System 2015-04-08 06:58:15 UTC
rubygem-ox-2.1.8-2.fc20 has been pushed to the Fedora 20 stable repository.

Comment 17 Fedora Update System 2015-04-21 19:16:28 UTC
rubygem-ox-2.1.8-2.fc22 has been pushed to the Fedora 22 stable repository.

Comment 18 Fedora Update System 2015-04-22 18:46:33 UTC
rubygem-ox-2.1.8-2.el7 has been pushed to the Fedora EPEL 7 stable repository.


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