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@xxxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review