Bug 241487
Summary: | Review Request: cduce - An XML-oriented functional language (ocaml) | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Richard W.M. Jones <rjones> |
Component: | Package Review | Assignee: | Xavier Lamien <lxtnow> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | gc |
Target Milestone: | --- | Flags: | lxtnow:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-03-28 19:08:00 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 240557, 241472, 241475, 241476, 241483, 241486 | ||
Bug Blocks: |
Description
Richard W.M. Jones
2007-05-26 15:37:48 UTC
Seems this has a dependency on ocaml-ocamlnet-devel which isn't in the distro yet. In fact, it hasn't been reviewed yet. I'll add a dependency. i'll start a full review once the bug dependency #241472 review will done (by me). Added other bug dependencies blocks Upstream CDuce haven't released a version which works with OCaml 3.10 yet. Add magic word ocaml to summary. Any news about it? The CDuce team has released a version compatible with OCaml 3.10 three months ago I'll take a look, but it is an issue of spare time at the moment. Spec URL: http://annexia.org/tmp/ocaml/cduce.spec SRPM URL: http://annexia.org/tmp/ocaml/cduce-0.5.0-1.src.rpm * Thu Sep 13 2007 Richard W.M. Jones <rjones> - 0.5.0-1 - Upstream version 0.5.0 - Remove RPATH in cduce binary. - ExcludeArch ppc64. - Build without ocamlopt. - Convert AUTHORS file to UTF-8. - Ignore Parsetree module. ok, I'll check this next week ;) I tried it on a fresh Fedora 8 install plus ocaml-ocamlnet-2.2.9-1.fc8.i386.rpm ocaml-ocamlnet-devel-2.2.9-1.fc8.i386.rpm ocaml-pxp-1.2.0test1-2.fc8.i386.rpm ocaml-pxp-devel-1.2.0test1-2.fc8.i386.rpm It works like a charm. The only think I do not understand is why you renamed the ocaml cgi package to netcgi1. This for instance makes it impossible to compile cduce from svn without appying first the patch. In any case thanks a lot for the great work. Looking forward to see this package approved I've just rechecked and this still builds. Any news? A new 0.5.1 version of CDuce has been released. http://www.cduce.org/download.html Without the netcgi1 patch I can't get cduce to compile: Checking for package camlp4... ok Checking for package num... ok Checking for package pcre... ok Checking for package ulex... ok Checking for package cgi... ocamlfind: Package `cgi' not found failed ! *** Fatal error: Required package is not available error: Bad exit status from /var/tmp/rpm-tmp.44682 (%prep) Spec URL: http://annexia.org/tmp/ocaml/cduce.spec SRPM URL: http://annexia.org/tmp/ocaml/cduce-0.5.1-1.fc9.src.rpm * Fri Feb 15 2008 Richard W.M. Jones <rjones> - 0.5.1-1 - New upstream release 0.5.1. - New OCaml 3.10.1. (In reply to comment #13) > Without the netcgi1 patch I can't get cduce to compile: > > Checking for package camlp4... ok > Checking for package num... ok > Checking for package pcre... ok > Checking for package ulex... ok > Checking for package cgi... ocamlfind: Package `cgi' not found > failed ! Ok, we fixed it in the svn version. Now if cgi package is not found then cduce is build but dtd2cduce and and the web interface interpreter are not. Thanks for the bug report. (In reply to comment #14) > Spec URL: http://annexia.org/tmp/ocaml/cduce.spec > In cduce.spec the following lines mv AUTHORS AUTHORS.old iconv -f ISO-8859-1 -t UTF-8 < AUTHORS.old > AUTHORS are no longer useful since now the AUTHORS file is in UTF-8 (In reply to comment #15) > (In reply to comment #13) > > Without the netcgi1 patch I can't get cduce to compile: > > > > Checking for package camlp4... ok > > Checking for package num... ok > > Checking for package pcre... ok > > Checking for package ulex... ok > > Checking for package cgi... ocamlfind: Package `cgi' not found > > failed ! > > Ok, we fixed it in the svn version. Now if cgi package is not found > then cduce is build but dtd2cduce and and the web interface interpreter > are not. Thanks for the bug report. It now uses the netcgi1 package provided by ocaml-ocamlnet-xxx.rpm Any idea when a new upstream official release might be out? When one comes along I'll produce a new Fedora package for review. (In reply to comment #18) > Any idea when a new upstream official release might be out? > Yes: today :-) http://www.cduce.org/download.html Is there a packaging problem with cduce 0.5.2? It seems like the file dtd2cduce.ml is missing, so when I build it I get this error: Pack cduce_lib.cmo Build cduce_lib.cma make[1]: Leaving directory `/home/rjones/rpmbuild/BUILD/cduce-0.5.2' make: *** No rule to make target `tools/dtd2cduce.ml', needed by `dtd2cduce'. Stop. error: Bad exit status from /var/tmp/rpm-tmp.31858 (%build) Below is a diff of the manifests for 0.5.1 and 0.5.2: @@ -51,6 +51,8 @@ ./LICENSE ./Makefile ./Makefile.conf.template +./Makefile.macosx +./make-package-macosx ./META.in ./misc/bool.ml ./misc/bool.mli @@ -146,10 +148,6 @@ ./schema/schema_xml.ml ./schema/schema_xml.mli ./schema/TODO -./tools/.cvsignore -./tools/dtd2cduce.ml -./tools/pull.ml -./tools/validate.ml ./types/atoms.ml ./types/atoms.mli ./types/boolean.ml Here's a new release based on 0.5.2-1. Please don't use '-' character in upstream release numbers! It makes it hard to package! Anyway: Spec URL: http://www.annexia.org/tmp/ocaml/cduce.spec SRPM URL: http://www.annexia.org/tmp/ocaml/cduce-0.5.2.1-1.fc9.src.rpm * Tue Feb 26 2008 Richard W.M. Jones <rjones> - 0.5.2.1-1 - New upstream release 0.5.2-1. Here are the rpmlint errors: $ rpmlint /home/rjones/rpmbuild/SRPMS/cduce-0.5.2.1-1.fc9.src.rpm cduce.src:63: E: configure-without-libdir-spec This is a problem in rpmlint. See bug 433783. $ rpmlint /home/rjones/rpmbuild/RPMS/i386/cduce-0.5.2.1-1.fc9.i386.rpm cduce.i386: E: only-non-binary-in-usr-lib cduce.i386: W: ocaml-naming-policy-not-applied /usr/lib/ocaml/cduce/cduce_lib.cmi See also bug 433783. $ rpmlint /home/rjones/rpmbuild/RPMS/i386/cduce-devel-0.5.2.1-1.fc9.i386.rpm cduce-devel.i386: W: ocaml-naming-policy-not-applied /usr/lib/ocaml/cduce/cduce_lib.a As above. just approved lastest require of ocaml-cduce. Will finalize this one once you will import and build ocaml-pxp as well. ocaml-pxp is built now. Hello Xavier, any news on this? We should be able to approve it now? yep it is, sorry for the delay. Fails on missing libs : --> http://koji.fedoraproject.org/koji/getfile?taskID=529687&name=build.log Ugh, missing BR. Fixed and test built in Koji here: http://koji.fedoraproject.org/koji/taskinfo?taskID=530193 Spec URL: http://www.annexia.org/tmp/ocaml/cduce.spec SRPM URL: http://www.annexia.org/tmp/ocaml/cduce-0.5.2.1-4.fc9.src.rpm * Tue Mar 25 2008 Richard W.M. Jones <rjones> - 0.5.2.1-4 - ExcludeArch ppc64 again (see bug 438843). * Tue Mar 25 2008 Richard W.M. Jones <rjones> - 0.5.2.1-3 - Added BR pcre-devel. - Test build in Koji. * Wed Mar 5 2008 Richard W.M. Jones <rjones> - 0.5.2.1-2 - Remove ExcludeArch ppc64. test pass. This package is ** Approved **. New Package CVS Request ======================= Package Name: cduce Short Description: CDuce is a modern XML-oriented functional language Owners: rjones Branches: F-8 InitialCC: rjones Cvsextras Commits: yes Great! cvs done. Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=536542 Fails in F-8 at the moment because ocaml-pxp hasn't yet migrated out to the main repository, but I don't anticipate any problems. |