Bug 908357 (Horde_Xml_Wbxml) - Review Request: php-horde-Horde-Xml-Wbxml - Provides an API for encoding and decoding WBXML documents
Summary: Review Request: php-horde-Horde-Xml-Wbxml - Provides an API for encoding and ...
Keywords:
Status: CLOSED ERRATA
Alias: Horde_Xml_Wbxml
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Shawn Iwinski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: horde-xml-wbxml (view as bug list)
Depends On: Horde_Util Horde_Test
Blocks: Horde_SyncMl
TreeView+ depends on / blocked
 
Reported: 2013-02-06 13:59 UTC by Remi Collet
Modified: 2013-03-26 13:04 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-27 18:02:23 UTC
shawn: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
php-horde-Horde-Xml-Wbxml-review.txt (6.88 KB, text/plain)
2013-02-10 04:42 UTC, Shawn Iwinski
no flags Details
phpci.log (12.49 KB, text/x-log)
2013-02-10 04:42 UTC, Shawn Iwinski
no flags Details

Comment 1 Remi Collet 2013-02-06 14:00:19 UTC
*** Bug 785481 has been marked as a duplicate of this bug. ***

Comment 2 Shawn Iwinski 2013-02-10 04:42:21 UTC
Created attachment 695663 [details]
php-horde-Horde-Xml-Wbxml-review.txt

Generated by fedora-review 0.3.1 (b71abc1) last change: 2012-10-16
Buildroot used: fedora-rawhide-x86_64
Command line :/usr/bin/fedora-review --mock-config fedora-rawhide-x86_64 --bug 908357

Comment 3 Shawn Iwinski 2013-02-10 04:42:50 UTC
Created attachment 695664 [details]
phpci.log

Comment 4 Shawn Iwinski 2013-02-10 04:54:48 UTC
Couldn't the "> 12" be removed from "%if 0%{?fedora} > 12"?  Or even just changed to "%{?fedora:BuildRequires: libwbxml}"?



===== MUST items =====

[!]: Requires correct, justified where necessary.
     
     phpci: pcre, xml

Comment 5 Remi Collet 2013-02-10 06:10:27 UTC
Missing requires:
https://github.com/remicollet/remirepo/commit/9c61bb7e79ed70782d80b537174436c55b3b1ded

> Couldn't the "> 12" be removed ...
I prefer to keep it (which makle backport easier) and because I don't know yet if this package will be available in RHEL-7, so could become
   %if 0%{?fedora} > 12 || 0%{?rhel} > 6

Spec: https://raw.github.com/remicollet/remirepo/9c61bb7e79ed70782d80b537174436c55b3b1ded/php/horde/php-horde-Horde-Xml-Wbxml/php-horde-Horde-Xml-Wbxml.spec
Srpm: http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Xml-Wbxml-2.0.1-3.remi.src.rpm

Comment 6 Shawn Iwinski 2013-02-10 17:15:05 UTC
[!]: Requires correct, justified where necessary.  =>  FIXED

No blockers.

===== APPROVED =====

Comment 7 Remi Collet 2013-02-10 17:46:51 UTC
Thanks for the review

New Package SCM Request
=======================
Package Name: php-horde-Horde-Xml-Wbxml
Short Description: Provides an API for encoding and decoding WBXML documents
Owners: remi nb
Branches: f18 el6
InitialCC:

Comment 8 Gwyn Ciesla 2013-02-11 13:55:45 UTC
Git done (by process-git-requests).

Comment 9 Fedora Update System 2013-02-11 15:03:10 UTC
php-horde-Horde-Xml-Wbxml-2.0.1-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-horde-Horde-Xml-Wbxml-2.0.1-3.fc18

Comment 10 Fedora Update System 2013-02-11 15:03:51 UTC
php-horde-Horde-Xml-Wbxml-2.0.1-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-horde-Horde-Xml-Wbxml-2.0.1-3.el6

Comment 11 Fedora Update System 2013-02-12 05:06:53 UTC
php-horde-Horde-Xml-Wbxml-2.0.1-3.fc18 has been pushed to the Fedora 18 testing repository.

Comment 12 Fedora Update System 2013-02-27 18:02:35 UTC
php-horde-Horde-Xml-Wbxml-2.0.1-3.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 13 Fedora Update System 2013-02-28 06:55:20 UTC
php-horde-Horde-Xml-Wbxml-2.0.1-3.fc18 has been pushed to the Fedora 18 stable repository.


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