[Bug 253571] Review Request: ocaml-fileutils - OCaml library for common file and filename operations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]