[Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code


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





------- Additional Comments From rjones at redhat.com  2008-03-01 08:25 EST -------
> This is yet another package that doesn't specify any version of the LGPL.  How
> did you determine that it's not LGPLv2+?  I thought that linking with ocaml
> libraries might force the issue, but the base ocaml package in rawhide doesn't
> even have a valid license tag so that's no help.

Disclaimer: I'm upstream maintainer of this package too.

The license is the famous LGPLv2+ with exceptions.  I've made a
new upstream package which adds proper license information to
the head of each file.
(http://merjis.com/developers/perl4caml/)

> Does this package handle the perl modules it wraps not being installed at
> builr or runtime?  It still seems to build the Net::Google stuff, for example,
> even though that package isn't required at all.  And that begs the question of
> whether those perl packages are actually needed at runtime.  The package
> builds without them and I didn't see any differences in the resulting package.

No it doesn't need the Perl package at build time.

At run time also it will work unless of course you actually try to use
the specific wrapped module, in which case you'll get a runtime error
from Perl when it tries to 'use Net::Google' (or whichever).  The
basic runtime functionality requires only libperl.so, for which there
is an automatically added requires.

----

Here's a new package:

Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-perl4caml.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-perl4caml-0.9.5-1.fc9.src.rpm

* Sat Mar  1 2008 Richard W.M. Jones <rjones at redhat.com> - 0.9.5-1
- New upstream release 0.9.5.
- Clarify license is LGPLv2+ with exceptions
- Remove excessive BuildRequires - Perl modules not needed for building.
- Pass RPM C flags to the make.
- 'make test' fails where perl4caml is already installed.


-- 
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, or are watching someone who is.