Bug 241476 - Review Request: ocaml-pxp - Validating XML parser
Summary: Review Request: ocaml-pxp - Validating XML parser
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Xavier Lamien
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 240557 241472 241475
Blocks: 241487
TreeView+ depends on / blocked
 
Reported: 2007-05-26 13:07 UTC by Richard W.M. Jones
Modified: 2008-03-02 11:09 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-03-02 11:09:19 UTC
Type: ---
Embargoed:
lxtnow: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Richard W.M. Jones 2007-05-26 13:07:12 UTC
Spec URL: http://annexia.org/tmp/ocaml-pxp.spec
SRPM URL: http://annexia.org/tmp/ocaml-pxp-1.2.0test1-1.src.rpm
Description: PXP is a validating XML parser library for OCaml.  Main site
 is: http://www.ocaml-programming.de/packages/ (scroll down a bit)

Comment 1 Xavier Lamien 2007-06-09 23:54:03 UTC
Added bug dependencies blocks which must be review first.


Comment 2 Richard W.M. Jones 2007-06-11 19:56:19 UTC
Upstream PXP haven't released a version which works with OCaml 3.10 yet.

Comment 3 Richard W.M. Jones 2007-09-13 16:04:20 UTC
Spec URL: http://annexia.org/tmp/ocaml/ocaml-pxp.spec
SRPM URL: http://annexia.org/tmp/ocaml/ocaml-pxp-1.2.0test1-2.src.rpm

* Thu Sep 13 2007 Richard W.M. Jones <rjones> - 1.2.0test1-2
- Build on OCaml 3.10
- Disable building the preprocessor (requires old camlp4 or camlp5).
- License is BSD.
- Ignore Parsetree.


Comment 4 Xavier Lamien 2007-09-21 15:59:15 UTC
ok, 
I'll check this next week ;)

Comment 5 Giuseppe Castagna 2008-01-13 07:28:41 UTC
Any progress on this bug?  It blocks bug #241487

Comment 6 Xavier Lamien 2008-01-15 21:50:20 UTC
Build failed on rawhide.
Some packages are missing on this one :
---------------------------------
DEBUG util.py:261:  No Package Found for ocaml-ulex-devel
DEBUG util.py:261:  No Package Found for ocaml-findlib-devel
DEBUG util.py:261:  No Package Found for ocaml-ocamldoc
DEBUG util.py:261:  No Package Found for ocaml-ocamlnet-devel
DEBUG util.py:261:  No Package Found for ocaml
--------------------------------

And on f-8, this following is missing :
--------------------------------
DEBUG util.py:261:  No Package Found for ocaml-ocamlnet-devel
--------------------------------

Comment 7 Richard W.M. Jones 2008-01-16 12:41:47 UTC
I built ocaml-ocamlnet for F8, but didn't get around to working out to
use Bodhi to actually push it into F8.  If you want to push it to F8 or
tell me how to do it please be my guest.

Nevertheless, I did build all of the above for devel so they should be
there.  I don't have a rawhide machine to test on right now, and
word on the street is that rawhide doesn't install at the moment.

Comment 8 Xavier Lamien 2008-01-17 15:33:18 UTC
done.

Need some approval to be validate from bodhi to be pushed to stable.
And then, will re-do a koji build.

Comment 9 Richard W.M. Jones 2008-02-12 12:31:24 UTC
There was also a missing BR on pcre-devel and ocaml-pcre-devel.  New version
fixes those:

Spec URL: http://annexia.org/tmp/ocaml/ocaml-pxp.spec
SRPM URL: http://annexia.org/tmp/ocaml/ocaml-pxp-1.2.0test1-4.fc8.src.rpm

* Tue Feb 12 2008 Richard W.M. Jones <rjones> - 1.2.0test1-4
- Added BR ocaml-pcre-devel, pcre-devel

* Thu Sep 13 2007 Richard W.M. Jones <rjones> - 1.2.0test1-3
- ExcludeArch ppc64

NB. The remaining rpmlint warnings are all bogus because it doesn't
know that *.cmi and *.cma files are really binaries.

Comment 10 Xavier Lamien 2008-02-12 20:54:59 UTC
hm...new release of Ocaml seem to be required.
http://koji.fedoraproject.org/koji/getfile?taskID=419865&name=build.log

Already built it on builders ?


Comment 11 Richard W.M. Jones 2008-02-15 19:00:42 UTC
That was a missing BR (ocaml-camlp4-devel).

I have checked this with the new OCaml in Rawhide (3.10.1) and it builds,
so here we go ...

Spec URL: http://annexia.org/tmp/ocaml/ocaml-pxp.spec
SRPM URL: http://annexia.org/tmp/ocaml/ocaml-pxp-1.2.0test1-5.fc9.src.rpm


Comment 12 Richard W.M. Jones 2008-02-24 19:11:17 UTC
My comment 9

  NB. The remaining rpmlint warnings are all bogus because it doesn't
  know that *.cmi and *.cma files are really binaries.

is wrong, but it's not a problem in the package.  See bug 433783.

Comment 13 Xavier Lamien 2008-03-01 17:57:00 UTC
Okay,

Build of ocaml-pxp-1.2.0test1-5.fc9.src.rpm successed.
-> http://koji.fedoraproject.org/koji/taskinfo?taskID=482389

The rpmlint output can be ignored regarding to the OCaml Packaging Guideline.

============
* APPROVED *
============

Comment 14 Richard W.M. Jones 2008-03-01 18:40:20 UTC
New Package CVS Request
=======================
Package Name: ocaml-pxp
Short Description: Validating XML parser
Owners: rjones
Branches: F-8
InitialCC: rjones
Cvsextras Commits: yes

Comment 15 Kevin Fenzi 2008-03-02 02:39:29 UTC
cvs done.


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