[Bug 253571] Review Request: ocaml-fileutils - OCaml library for common file and filename operations
bugzilla at redhat.com
bugzilla at redhat.com
Sat Feb 16 22:20:00 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-fileutils - OCaml library for common file and filename operations
https://bugzilla.redhat.com/show_bug.cgi?id=253571
------- Additional Comments From tibbs at math.uh.edu 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.
More information about the package-review
mailing list