[Bug 460894] Review Request: ocsigen - Web programming framework

2009-08-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(rjo...@redhat.com
   ||)




--- Comment #16 from Richard W.M. Jones rjo...@redhat.com  2009-08-05 
03:35:53 EDT ---
I'll set it to NEEDINFO of me, but I don't have much time
at the moment to spend on this.  I'm quite happy leaving these
bugs open for as long as they take, or if someone else is
motivated to pick up the package and finish the review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-08-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

  Status Whiteboard||NotReady




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-08-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #15 from Jason Tibbitts ti...@math.uh.edu  2009-08-03 21:42:38 
EDT ---
Richard, did you want to update the package and post fresh links here?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-08-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #14 from Stéphane Glondu st...@glondu.net  2009-08-01 07:57:30 
EDT ---
Comment #11 is obsolete with OCaml 3.11.1.

The ocaml-mixed-executable shouldn't be there whith latest ocsigen (1.2) and
ocaml-ssl with the following patch:

http://patch-tracking.debian.net/patch/series/view/ocaml-ssl/0.4.3-2/makefile

(committed in rev. 6455 of upstream SVN).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-07-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #13 from Jason Tibbitts ti...@math.uh.edu  2009-07-31 22:41:04 
EDT ---
What's the status of this ticket?  Given comment #11, I'm not sure if this is
ready for review, but I built the package from comment #4 on rawhide and it
seems to at least build and install OK.  (I've zero idea how to test it.)

One thing I noticed is that the %pre scriptlet references %contentdir, which
isn't defined so it gets inserted literally into the scriptlet which almost
certainly doesn't do what's expected, and also causes:
  ocsigen.x86_64: W: percent-in-%pre
   
  There's also
  ocsigen.x86_64: W: ocaml-mixed-executable
/usr/lib64/ocsigen/extensions/ocsidbm
  ocsigen.x86_64: W: ocaml-mixed-executable /usr/bin/ocsigen
which I've not seen before.  rpmlint says that nothing should build with ocamlc
-custom.  I don't see '-custom' anywhere in the build log so I'm not sure
what's up there.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-03-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #12 from Richard W.M. Jones rjo...@redhat.com  2009-03-10 
11:16:03 EDT ---
*** Bug 460841 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Stéphane Glondu st...@glondu.net changed:

   What|Removed |Added

 CC||st...@glondu.net




--- Comment #11 from Stéphane Glondu st...@glondu.net  2009-01-21 12:09:09 
EDT ---
There might be problems building with OCaml 3.11.0, see:

  http://caml.inria.fr/mantis/view.php?id=4699

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #5 from Wes Shull wes.sh...@gmail.com  2009-01-14 03:45:35 EDT ---
I can verify it now builds, installs, and runs (only did basic testing) without
error for me on f10-x86_64; thanks Richard.

A few minor points:

* /usr/bin/ocsigen is system_u:object_r:bin_t; do we maybe want to make it
system_u:object_r:httpd_exec_t like apache httpd since it's just another
webserver?

* I got no -debuginfo package out of the build, and /usr/bin/ocsigen is not
stripped.

* The ocsigen quickstart page indicates most distros include an initscript,
which we seem to be lacking.  I'd volunteer but shell scripts make me want to
gouge my eyes out.

Apart from these potential considerations, looks good to me.  Should I start
rabble-rousing on the -devel list to get some real certified reviewers on this?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #7 from Richard W.M. Jones rjo...@redhat.com  2009-01-14 05:34:46 
EDT ---
(In reply to comment #6)
  * The ocsigen quickstart page indicates most distros include an initscript,
  which we seem to be lacking.  I'd volunteer but shell scripts make me want 
  to
  gouge my eyes out.
 
 Probably best to copy the one from Debian, if they have one:
 
 http://git.debian.org/?p=pkg-ocaml-maint/packages/ocsigen.git;a=summary

Here we go, smimram wrote one:

http://git.debian.org/?p=pkg-ocaml-maint/packages/ocsigen.git;a=blob_plain;f=debian/ocsigen.init;hb=HEAD

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #8 from Richard W.M. Jones rjo...@redhat.com  2009-01-14 06:00:55 
EDT ---
About ocamlduce, see bug 479970.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #6 from Richard W.M. Jones rjo...@redhat.com  2009-01-14 05:33:36 
EDT ---
(In reply to comment #5)
 I can verify it now builds, installs, and runs (only did basic testing) 
 without
 error for me on f10-x86_64; thanks Richard.
 
 A few minor points:
 
 * /usr/bin/ocsigen is system_u:object_r:bin_t; do we maybe want to make it
 system_u:object_r:httpd_exec_t like apache httpd since it's just another
 webserver?

I have absolutely no idea how to do this, but sure it sounds
like a good idea.

 * I got no -debuginfo package out of the build, and /usr/bin/ocsigen is not
 stripped.

Debuginfo isn't really that useful for OCaml programs, since
gdb doesn't know about OCaml.  So you end up having to debug
assembler.

 * The ocsigen quickstart page indicates most distros include an initscript,
 which we seem to be lacking.  I'd volunteer but shell scripts make me want to
 gouge my eyes out.

Probably best to copy the one from Debian, if they have one:

http://git.debian.org/?p=pkg-ocaml-maint/packages/ocsigen.git;a=summary

 Apart from these potential considerations, looks good to me.  Should I start
 rabble-rousing on the -devel list to get some real certified reviewers on 
 this?

Sure thing!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #9 from Wes Shull wes.sh...@gmail.com  2009-01-14 06:38:56 EDT ---
(In reply to comment #6)
 (In reply to comment #5)
  * /usr/bin/ocsigen is system_u:object_r:bin_t; do we maybe want to make it
  system_u:object_r:httpd_exec_t like apache httpd since it's just another
  webserver?
 
 I have absolutely no idea how to do this, but sure it sounds
 like a good idea.

bug 243302 details another package dealing with the same thing (yay for bz
being indexed by google now)

I do have a reasonable level of familiarity with creating selinux modules, so
unless you've nothing else to do, leave it for me and I'll take a poke at it
tomorrow.  Once we've got something tested/working we can run it by dwalsh to
make sure it's not insane.

 Debuginfo isn't really that useful for OCaml programs, since
 gdb doesn't know about OCaml.  So you end up having to debug
 assembler.

I take it the symbols are of use to ocamldebug then?  Or could we just strip
useless baggage and discard?

  * The ocsigen quickstart page indicates most distros include an initscript,
  which we seem to be lacking.  I'd volunteer but shell scripts make me want 
  to
  gouge my eyes out.
 
 Probably best to copy the one from Debian, if they have one:
 
 http://git.debian.org/?p=pkg-ocaml-maint/packages/ocsigen.git;a=summary

I can handle that level of copy-and-tweak, again unless you've just nothing
better to do.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #10 from Richard W.M. Jones rjo...@redhat.com  2009-01-14 
07:19:58 EDT ---
(In reply to comment #9)
  Debuginfo isn't really that useful for OCaml programs, since
  gdb doesn't know about OCaml.  So you end up having to debug
  assembler.
 
 I take it the symbols are of use to ocamldebug then?  Or could we just strip
 useless baggage and discard?

In ordinary Fedora C packages, foo-debuginfo contains the unstripped
object files (ie. containing DWARF debugging info).  The base
package ('foo') is stripped of course.  However the programs in
the base package contain a special buildid section, which, with a
suitably modified version of gdb, causes gdb to get the right
symbols out of the corresponding foo-debuginfo file.

All well and good for ordinary C/C++ programs.  However OCaml has
its own native code generator (ie. it generates machine code directly).
This code generator doesn't embed any DWARF debugging info, and as
a result if you try to generate a ocaml-foo-debuginfo subpackage
it will usually be empty.  (Except in some special cases, ie.
where C code is linked in to the program).  Thus we disable
generation of ocaml-foo-debuginfo subpackages as a matter of policy.

There's a separate issue which is that gdb is still able to
find ordinary symbols in unstripped OCaml executables, but because
gdb doesn't have DWARF info and in any case doesn't know anything
about the OCaml language, all it can do is show symbol locations
in assembler.  Believe it or not, I have debugged OCaml programs
on many occasions using gdb like this ...

ocamldebug is quite separate.  Firstly it only works on bytecode-
compiled OCaml programs.  Secondly, OCaml bytecode contains its
own debugging information, not related to and totally incompatible
with DWARF, which is only understood by ocamldebug and a few other
programs like ocamlprof.

Some enterprising person with LOTS of time on their hands needs
to add DWARF output to the OCaml native code generator(s), and
add support for the OCaml language in gdb.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #4 from Richard W.M. Jones rjo...@redhat.com  2009-01-13 09:27:56 
EDT ---
Here's an updated package which fixes everything in comment 3
and builds in Koji:

Spec URL: http://www.annexia.org/tmp/ocaml/ocsigen.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocsigen-1.1.0-8.fc11.src.rpm

* Tue Jan 13 2009 Richard W.M. Jones rjo...@redhat.com - 1.1.0-8
- Add some missing BRs (thanks Wes Shull).
- Make sure examples, extensions get installed in _libdir.
- ExcludeArch ppc64.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1049581

Note that we cannot yet build ocamlduce for OCaml 3.11.0 as we
are waiting for upstream to release a new version:
http://www.cduce.org/ocaml_install.html
Since I don't really use ocsigen at the moment, I can't say
whether this gravely affects the functionality of this package.
However it does compile OK without ocamlduce.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2009-01-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Wes Shull w...@kuoi.asui.uidaho.edu changed:

   What|Removed |Added

 CC||w...@kuoi.asui.uidaho.edu




--- Comment #3 from Wes Shull w...@kuoi.asui.uidaho.edu  2009-01-13 00:17:40 
EDT ---
Seeing how the Ocsigen home page says 02/09/08 Ocsigen and Lwt in Fedora,
maybe we ought to work on making it true?

I just tried to build the srpm on a local f10-x86_64 box (w/all current
updates) with no success.  I did ferret out a few missing BuildRequires:

ocaml-camlp4-devel for /usr/bin/camlp4
ocaml-ocamlnet-devel for -lnetaccel_c
ocaml-pcre-devel for -lpcre_stubs

Now it gets all the way past %doc and dies with:
File not found:
/home/wes/rpmbuild/BUILDROOT/ocsigen-1.1.0-5.fc10.x86_64/usr/lib64/ocsigen

Looking at the build output, I see a Checking for ocamlduce... not found, but
I see no ocamlduce in the repo (I do have cduce installed of course).

also
diff: xHTML.mli: No such file or directory

and later
ocamlyacc http_parser.mly
4 shift/reduce conflicts.

But I have no immediate idea what to do with that.

Does someone who actually knows the ocaml buildsystem (Richard?) want to take
another stab at this?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2008-09-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #1 from Jason Tibbitts [EMAIL PROTECTED]  2008-09-05 17:31:38 EDT 
---
This fails to build for me in rawhide.  The failure is long so I thought I'd do
a scratch build:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=809571
but that proceeded to fail because cduce-devel is not available on ppc64.  So
here's a scratch build on x86_64 only:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=809581

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2008-09-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894





--- Comment #2 from dex [EMAIL PROTECTED]  2008-09-05 21:55:17 EDT ---
Looks like a build dependency on ocaml-camlp4.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2008-09-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Bug 460894 depends on bug 460893, which changed state.

Bug 460893 Summary: Review Request: ocaml-lwt - OCaml lightweight thread library
https://bugzilla.redhat.com/show_bug.cgi?id=460893

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460894] Review Request: ocsigen - Web programming framework

2008-09-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=460894


Richard W.M. Jones [EMAIL PROTECTED] changed:

   What|Removed |Added

 Depends on||460893
Customer Facing||---




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review