[Bug 434630] New: Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
10:17 a.m.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=434630
Summary: Review Request: ocaml-perl4caml - OCaml library for
calling Perl libraries and code
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: rjones(a)redhat.com
QAContact: extras-qa(a)fedoraproject.org
CC: fedora-package-review@redhat.com,notting(a)redhat.com
Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-perl4caml.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-perl4caml-0.9.4-1.fc9.src.rpm
Description: OCaml library for calling Perl libraries and code
Perl4caml allows you to use Perl code within Objective CAML (OCaml),
thus neatly side-stepping the (old) problem with OCaml which was that
it lacked a comprehensive set of libraries. Well now you can use any
part of CPAN in your OCaml code.
--
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.
10:21 a.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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(a)redhat.com 2008-02-23 11:21 EST -------
The package is rpmlint-clean except for one odd warning against the -devel
package:
$ rpmlint ocaml-perl4caml-devel-0.9.4-1.fc9.i386.rpm
ocaml-perl4caml-devel.i386: W: binaryinfo-readelf-failed
I dug into this and found that it is because a '.a' file inside
the package contains another '.a' file. 'readelf' doesn't know
how to handle this situation:
$ ar t /usr/lib/ocaml/perl/libperl4caml.a
perl_c.o
DynaLoader.a
$ readelf -a /usr/lib/ocaml/perl/libperl4caml.a
[...]
readelf: Error: Unable to read in 0x2020 bytes of section headers
File: /usr/lib/ocaml/perl/libperl4caml.a(DynaLoader.a)
readelf: Error: Not an ELF file - it has the wrong magic bytes at the start
However the library file links correctly, so it seems like its not
a warning we need to worry about.
--
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.
5:30 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
rjones(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-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, or are watching someone who is.
1:06 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
rjones(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-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, or are watching someone who is.
2:07 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
tibbs(a)math.uh.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|nobody(a)fedoraproject.org |tibbs(a)math.uh.edu
Status|NEW |ASSIGNED
Flag| |fedora-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, or are watching someone who is.
3:22 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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 tibbs(a)math.uh.edu 2008-02-29 16:22 EST -------
That rpmlint complaint is indeed weird, but I agree with your assessment that
it's not problematic.
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.
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.
I checked the build log for gcc calls and I noticed that the compiler isn't
being called with the proper set of flags. It looks like the flags are
specified directly in the Makefile. They should probably match up with
%optflags although I'm not sure what would happen if -g was passed since
debuginfo isn't being generated.
* source files match upstream:
56f3f6e5cb02dfb9f319cdc1c761586d573c6ecc14639e186a9783d253934d3d
perl4caml-0.9.4.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* ocaml packaging guidelies are followed.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
? license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
? BuildRequires may be excessive.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint has acceptable complaints.
* final provides and requires are sane:
ocaml-perl4caml-0.9.4-1.fc9.x86_64.rpm
dllperl4caml.so()(64bit)
ocaml(Perl) = 62ffc8b090d3554adbdf29a2b91809b0
ocaml(Pl_Data_Dumper) = 46141594dfbf6ad5d7f8fe499dbb5602
ocaml(Pl_Date_Calc) = 81309d39376fa4c47fd03e919dd278df
ocaml(Pl_Date_Format) = a08a10d6da6a544612bd0ffdb359f18a
ocaml(Pl_Date_Parse) = 44f3f7ef90b613ae8400caa859bf403d
ocaml(Pl_HTML_Element) = 5db46ddac9327df9be091e1aa5a55892
ocaml(Pl_HTML_Form) = 8d3305d588ce5543dc6f0b5ee217c3e9
ocaml(Pl_HTML_Parser) = df42b14d7519e0affaf2260777e063e6
ocaml(Pl_HTML_TreeBuilder) = ca4eb08fe160b9e7baf34592ccf04ad8
ocaml(Pl_HTTP_Cookies) = 14c83b6408189bbd38318b1cfb8fd0d8
ocaml(Pl_HTTP_Headers) = 19b0ed2af45c31f51c023967b9f6acaf
ocaml(Pl_HTTP_Message) = 3fb2d4309ab697c1fa0b2319e1af17c3
ocaml(Pl_HTTP_Request) = 5e5dba683d5c2996df9c44fd2f3571fd
ocaml(Pl_HTTP_Request_Common) = 528fe8dfa8822dcd758a0187fee7b975
ocaml(Pl_HTTP_Response) = f92917d741a3502dcda657ff9a889610
ocaml(Pl_LWP_UserAgent) = c85c04042060d5c093f00109a54290ba
ocaml(Pl_Net_Google_Cache) = a0b7834c394b63bc91704beca3d3d122
ocaml(Pl_Net_Google) = d420a90f979db3e91eb6ba827a903be7
ocaml(Pl_Net_Google_Response) = b839208e102c02061f7268b0fa8da715
ocaml(Pl_Net_Google_Search) = 55086cc7adb3bd1b5acf687f8a5405db
ocaml(Pl_Net_Google_Spelling) = 24181dfa9009a3375340ca9631720cc8
ocaml(Pl_Template) = 492b2c675a77fe96b3867b6465d7397f
ocaml(Pl_URI) = 451cd0d14a52791c57df8b0e8da1cd18
ocaml(Pl_WWW_Mechanize) = a180ccd19ff5bb9cd910199d2926ac30
ocaml-perl4caml = 0.9.4-1.fc9
=
libperl.so()(64bit)
libutil.so.1()(64bit)
ocaml(Callback) = e5ca1fb5990fac2b7b17cbb1712cffe2
ocaml(CamlinternalOO) = 6d0d5b328d6db88f403ca4393b4abd38
ocaml(Int32) = 711321870c949bd3bbdd092d9bae92e4
ocaml(List) = da1ce9168f0408ff26158af757456948
ocaml(Nativeint) = e79cdc4d3575c2ed044955cb7ef49aca
ocaml(Obj) = 5cfae708052c692ea39d23ed930fd64d
ocaml(Pervasives) = 8ba3d1faa24d659525c9025f41fd0c57
ocaml(String) = 2c162ab314b2f0a2cfd22d471b2e21ab
ocaml(runtime) = 3.10.1
ocaml-perl4caml-devel-0.9.4-1.fc9.x86_64.rpm
perl(TestClass)
ocaml-perl4caml-devel = 0.9.4-1.fc9
=
ocaml-perl4caml = 0.9.4-1.fc9
* %check is present and all tests pass:
All tests succeeded.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files (except permitted COPYING.LIB file)
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
The actual documentation is ~80% of the -devel package, but it's not in the
main package so I have no problem with it.
* %docs are not necessary for the proper functioning of the package.
--
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.
7:25 a.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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(a)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(a)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.
12:43 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
tibbs(a)math.uh.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-review+
------- Additional Comments From tibbs(a)math.uh.edu 2008-03-02 13:43 EST -------
rpmlint has grown complaints about the license:
W: invalid-license LGPLv2+ with exceptions
but that's bogus.
This package seems fine to me now. I guess you could just delete the block of
commended out dependencies, and you should probably take the rpath thing to the
committee because my understanding was always that the issue was an rpath
containing one of the standard directories (like /usr/lib64) that caused the
main issue.
APPROVED
--
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.
7:53 a.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
rjones(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
------- Additional Comments From rjones(a)redhat.com 2008-03-03 08:53 EST -------
New Package CVS Request
=======================
Package Name: ocaml-perl4caml
Short Description: OCaml library for calling Perl libraries and code
Owners: rjones
Branches: F-8
InitialCC: rjones
Cvsextras Commits: yes
--
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.
2:05 p.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
kevin(a)tummy.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin(a)tummy.com 2008-03-03 15:05 EST -------
cvs done.
--
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.
3:19 a.m.
New subject: [Bug 434630] Review Request: ocaml-perl4caml - OCaml library for calling Perl libraries and code
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
rjones(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |CLOSED
Resolution| |RAWHIDE
------- Additional Comments From rjones(a)redhat.com 2008-03-04 04:19 EST -------
F-8: http://koji.fedoraproject.org/koji/taskinfo?taskID=489973
devel: http://koji.fedoraproject.org/koji/taskinfo?taskID=489978
--
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.
5957
days inactive
5967
days old
package-review@lists.fedoraproject.org
10 comments
1 participants
participants (1)
-
Red Hat Bugzilla