[Bug 434169] Review Request: ocaml-zip - OCaml library for reading and writing zip, jar and gzip files

bugzilla at redhat.com bugzilla at redhat.com
Thu Feb 28 19:42:11 UTC 2008


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-zip - OCaml library for reading and writing zip, jar and gzip files


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


tibbs at math.uh.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody at fedoraproject.org    |tibbs at math.uh.edu
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review+




------- Additional Comments From tibbs at math.uh.edu  2008-02-28 14:42 EST -------
This looks good to me.  Again they're not too clear about the LGPL version in
use, but there is some statement about the version in use at the top of the
LICENSE file, which seems good enough to me.

* source files match upstream:
   d45af5deee6126137eb1bd896373ceeabacb916e30b977122de2eb3134336f08  
   camlzip-1.03.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* package follows ocaml guidelines
* 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 are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
  ocaml-zip-1.03-1.fc9.x86_64.rpm
   dllcamlzip.so()(64bit)
   ocaml(Gzip) = 6258f07d6e9179c9ee17180766a0978e
   ocaml(Zip) = 48478f1ccd637a3af5ebf6d8031b475a
   ocaml(Zlib) = f525a2d075f92332c0da098f58f3b6fb
   ocaml-zip = 1.03-1.fc9
  =
   libz.so.1()(64bit)
   ocaml(Callback) = e5ca1fb5990fac2b7b17cbb1712cffe2
   ocaml(Char) = e98bc9c9e918a84b3c1a5a122d42fac1
   ocaml(Hashtbl) = 083f2c94b44ff4e0b3220aaea6a783b4
   ocaml(Int32) = 711321870c949bd3bbdd092d9bae92e4
   ocaml(Int64) = f8f7e2e4c0667ead94596040b12e732d
   ocaml(List) = da1ce9168f0408ff26158af757456948
   ocaml(Pervasives) = 8ba3d1faa24d659525c9025f41fd0c57
   ocaml(String) = 2c162ab314b2f0a2cfd22d471b2e21ab
   ocaml(Sys) = 0da495f5a80f31899139359805318f28
   ocaml(Unix) = 9a46a8db115947409e54686ada118599
   ocaml(runtime) = 3.10.1

  ocaml-zip-devel-1.03-1.fc9.x86_64.rpm
   ocaml-zip-devel = 1.03-1.fc9
  =
   ocaml-zip = 1.03-1.fc9

* %check is not present; no test suite upstream.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files (except allowed license file)
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

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.




More information about the Fedora-package-review mailing list