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-fileutils - OCaml library for common file and filename operations https://bugzilla.redhat.com/show_bug.cgi?id=253571 ------- Additional Comments From tibbs@xxxxxxxxxxx 2008-02-16 17:19 EST ------- I figured this wouldn't build because of the ocaml update, but it turns out that it builds fine. rpmlint has only the two weird complaints above; someone who understands what ocaml is doing really needs to get with the rpmlint maintainer and figure out the source of these complaints. Maybe it just doesn't like the fact that the only thing directly in _libdir is a directory. There's a test suite there; I tried running it but it seems to need the package to be installed before it works, and I don't know if it's possible to adjust the various search paths to pick up the files from the build directory. There are several grammatical errors in the description which should be easy to fix up. I had hoped to find a better upstream URL than one pointing to a list of files for download, but the wiki that upstream has doesn't seem to have any actual content. As far as I can tell, the License is LGPLv2+ plus some type of exception, so I guess this is "LGPLv2+ with exceptions". In addition, the COPYING file needs to be added to the package. You should probably add AUTHOR, CHANGELOG and TODO as well. I'm not sure about the stuff in the "website" directory. I can't think of any better way to handle the broken build system of this package, and you've adequately commented things, so that's OK. The README file is duplicated between packages, but we already decided I wouldn't block on this. * source files match upstream: 4b3929f758dac11bebdcbb84963ab44e2a5ebf06023665dd511d61b19e9654b8 ocaml-fileutils-0.3.0.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * Follows ocaml packaging guidelines * summary is OK. X description could use some grammar fixes. * dist tag is present. * build root is OK. X license field does not match the actual license. * license is open source-compatible. X license text should be included in the package. * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly * rpmlint seems to have only bogus complaints. * final provides and requires are sane: ocaml-fileutils-0.3.0-2.fc9.x86_64.rpm ocaml(CygwinPath) = 55504b4f452dc9b72bcd7564a4c5a991 ocaml(CygwinPath_lexer) = 29edb3b5196b78e256fdcbafa566486b ocaml(CygwinPath_parser) = d7c4325a803605bd4737615702dcb060 ocaml(FilePath) = 80389f0c61bde05b590de9a464aa1831 ocaml(FilePath_type) = b20c6b0e73bf51cf8d25a2f256443dfc ocaml(FileUtil) = bd8589f4d97317061dd24d80db6eeacc ocaml(GenericPath_lexer) = d564536f943cee33d33f51880f85406d ocaml(GenericPath_parser) = 177c3bf79cd035b170e1d2efce3a36ff ocaml(MacOSPath) = 6eecbacbd1f32d7cbddb7f1e269358b7 ocaml(MacOSPath_lexer) = 473b7b6c310c611393a88e6922dfc897 ocaml(MacOSPath_parser) = 323e664f9989fd08de7097f80bc34995 ocaml(Str) = 56bb7ee61b2da83d42394686e3558fe4 ocaml(Unix) = 9a46a8db115947409e54686ada118599 ocaml(UnixLabels) = daaededf8793acd8b374c4e7c5d1fd16 ocaml(UnixPath) = 1538fbea534e0ac1e7f3a48c81330a6c ocaml(UnixPath_lexer) = 52138249abfd7fd19243bae955fec96a ocaml(UnixPath_parser) = 34b22f7becddd052a253243d516b2685 ocaml(Win32Path) = ef3947425c94dcbbbb4e80a851401ab0 ocaml(Win32Path_lexer) = 934bf4ebb71efb7a397e9fd8ce16a1b2 ocaml(Win32Path_parser) = b55b3f0165af59f53b11c55fa4f2495d ocaml-fileutils = 0.3.0-2.fc9 = ocaml(Array) = aa8e3cd5824f9bb40b93fcd38d0c95b5 ocaml(Buffer) = f6cef633ea14963b84b79c4095c63dc3 ocaml(Callback) = e5ca1fb5990fac2b7b17cbb1712cffe2 ocaml(Char) = e98bc9c9e918a84b3c1a5a122d42fac1 ocaml(Hashtbl) = 083f2c94b44ff4e0b3220aaea6a783b4 ocaml(Lexing) = b1793496643444d3762dd42bebe2cfe3 ocaml(List) = da1ce9168f0408ff26158af757456948 ocaml(Map) = dedde7683d54ae7db1eb97cc868dd047 ocaml(Obj) = 5cfae708052c692ea39d23ed930fd64d ocaml(Parsing) = 62cca107e4e88af303516459a87c3e9a ocaml(Pervasives) = 8ba3d1faa24d659525c9025f41fd0c57 ocaml(Printf) = 5dbbf45a03b54e6dbfcf39178d0d6341 ocaml(Set) = 7da14e671a035f12386ace3890018ef3 ocaml(Stream) = 21a833e12efd34ea0c87d8d9da959809 ocaml(String) = 2c162ab314b2f0a2cfd22d471b2e21ab ocaml(Sys) = 0da495f5a80f31899139359805318f28 ocaml(runtime) = 3.10.1 ocaml-fileutils-devel-0.3.0-2.fc9.x86_64.rpm ocaml-fileutils-devel = 0.3.0-2.fc9 = ocaml-fileutils = 0.3.0-2.fc9 * %check is not present; upstream test suite cannot easily be run. * no shared libraries are added to the regular linker search paths. * owns the directories it creates. * doesn't own any directories it shouldn't. ? README file is duplicated. * 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. -- 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