[Bug 477320] Review Request: ocaml-p3l - OCaml compiler for parallel programs

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts <tibbs at math.uh.edu> changed:

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




--- Comment #3 from Jason Tibbitts <tibbs at math.uh.edu>  2009-03-11 13:17:52 EDT ---
This looks fine, save for the duplicated LICENSE and README.Fedora files.  You
will need to pick one package for each file to live in.  The packaging
guidelines have recently become much clearer on this.

Is it possible (or reasonable) to run the tests in Examples/Tests at build
time?

I'm pretty sure that this is a syntax extension, so the .cmo files are needed.

* source files match upstream.  sha256sum:
   495e7231cf4cd17bf75405f5e5b507c0ff353349ec81866b3a851bd335c1eb18  
   ocamlp3l-2.03.tgz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* 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.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint is silent.
* final provides and requires are sane:
  ocaml-p3l-2.03-1.fc11.x86_64.rpm
   ocaml(Basedefs) = 4300eb5891d190ace73a8c30faa55e01
   ocaml(Command_options) = 3425ccf9d1bcf69c98bd6855fbf5d2d9
   ocaml(Commlib) = 73b6bbd27d3ee98dc1d99b753845efcc
   ocaml(Grafp3l) = ef66145601ce1c290f7f3909e656f407
   ocaml(Nodecode) = fcfbff84bc4c1e787e2190d072c60f0e
   ocaml(P3lstream) = 1e0b76796600f19bf3b4a3f7f8f590af
   ocaml(Parp3l) = 0a75d20f69876cdd7c7ac780ddae58e9
   ocaml(Seqp3l) = 5c01c19be0632fc803d831b01649d39f
   ocaml(Server) = 00e0a391fd00a6a4d91543daf344c029
   ocaml(Streams) = a4644f7bad89cbcd7631ec715002a54b
   ocaml(Template) = 1cfa9effbf934bd7300325692f0871b0
   ocaml(Version) = 01540deec20bd5eb701d54711b534ce7
   ocaml-p3l = 2.03-1.fc11
   ocaml-p3l(x86-64) = 2.03-1.fc11
  =
   ocaml(Arg) = b6513be035dc9c8a458c189cd8841700
   ocaml(Array) = 9c9fa5f11e2d6992c427dde4d1168489
   ocaml(Buffer) = 23af67395823b652b807c4ae0b581211
   ocaml(CamlinternalLazy) = ed280fb9736e9200aa47db73c5ff077f
   ocaml(Graphics) = fa6011251a7c1017decb62fd8af77d0d
   ocaml(Hashtbl) = ee2a3220e38a4350c5bc131ce9f3f6ce
   ocaml(Int32) = b2545c419b6b6a173cac4c0a3e7e0277
   ocaml(Int64) = d501d6e89fdce41c79f274fb464995d5
   ocaml(Lazy) = 4c7ed568fa7b5f73a2aa02eeb0e5e12b
   ocaml(List) = a0e2e49d266ff302f8667651a43f71ba
   ocaml(Marshal) = 02be0525cda0ca38ef8d49584e7769d6
   ocaml(Mutex) = 60fb057a4923c01932f593e58ab0bbac
   ocaml(Nativeint) = 7233ce5207a538fea4f0c61ed411ea2c
   ocaml(Obj) = c827f726ce05da709cf7de58fc15e324
   ocaml(Pervasives) = 88cb1505c8bdf9a4dcd2cdf3452732b4
   ocaml(Printexc) = fdf007941aa14d1a26323558012dbf52
   ocaml(Printf) = 807ecd3a1538992580464c03462c9964
   ocaml(Random) = 462fc826fd1ae9df8d15e3cb798cba9d
   ocaml(String) = ecc403546c1c50056801131811c39017
   ocaml(Sys) = 21bf525b2b3f3a46a54b96163adfe387
   ocaml(Thread) = da0ce921a4a489fa87637cc4648cf5e3
   ocaml(Unix) = 0596a58544f8cd88fed5bf5432a53d43
   ocaml(runtime) = 3.11.0

  ocaml-p3l-devel-2.03-1.fc11.x86_64.rpm
   ocaml-p3l-devel = 2.03-1.fc11
   ocaml-p3l-devel(x86-64) = 2.03-1.fc11
  =
   ocaml-p3l = 2.03-1.fc11

  ocaml-p3l-doc-2.03-1.fc11.x86_64.rpm
   ocaml-p3l-doc = 2.03-1.fc11
   ocaml-p3l-doc(x86-64) = 2.03-1.fc11
  =
   ocaml-p3l = 2.03-1.fc11

? %check is not present, but there seem to be some tests.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
X a few duplicates in %files.
* file permissions are appropriate.
* code, not content.
* large docs are in a separate -doc package.
* %docs are not necessary for the proper functioning of the package.
* .cma, .cmi, .so, .so.owner, META files in the main package.
* .a, .cmxa, .cmx and .mli files are in the -devel subpackage.
* .o and .ml files not included (.cmo fiels are OK as this is a syntax 
   extension)

-- 
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.