[Bug 445068] Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol

[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=445068





--- Comment #11 from Jason Tibbitts <tibbs at math.uh.edu>  2009-11-07 12:15:37 EDT ---
OK, this builds fine and rpmlint is silent

In multiple licensing scenarios, you'll need to add some indication of which
files are under which license.  The easiest way is to just add a comment to the
spec. 
http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Multiple_Licensing_Scenarios

You've duplicated the license files between -devel and the main package.  This
is clearly against the guidelines
  http://fedoraproject.org/wiki/Packaging:Guidelines#Duplicate_Files
but I know we've gone back and forth on this so I've requested that we get an
opinion from legal and get the guidelines modified if necessary.

The -devel package includes a .ml file; the OCaml guidelines say that these
shouldn't be  included except in specific circumstances.  I can't determine
from the information I have whether or not those circumstances apply here, and
there's no comment in the spec about it.

* source files match upstream.  sha256sum:
   1a8569a75195b35ec17348046b3aa8823f9f27e451214717692a86075cfbdb9b
   release-1.2.20.tar.bz2
* 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-bin-prot-1.2.20-1.fc12.x86_64.rpm
   dllbin_prot_stubs.so()(64bit)            
   ocaml(Bin_prot) = 68b4c083005ed9d5cde0767b0e6a9782
   ocaml(Pa_bin_prot) = d894b67ca8604bc43a9721e774c08303
   ocaml-bin-prot = 1.2.20-1.fc12                       
   ocaml-bin-prot(x86-64) = 1.2.20-1.fc12               
  =                                                  
   ocaml(Arg) = b6513be035dc9c8a458c189cd8841700        
   ocaml(Array) = 9c9fa5f11e2d6992c427dde4d1168489      
   ocaml(Bigarray) = fc2b6c88ffd318b9f111abe46ba99902   
   ocaml(Buffer) = 23af67395823b652b807c4ae0b581211     
   ocaml(Callback) = 71e1f9b7f211661f1dfeedab5ffae0cc   
   ocaml(CamlinternalLazy) = ed280fb9736e9200aa47db73c5ff077f
   ocaml(Camlp4) = bb930f7c2bed5d057c794fe07dc8596a          
   ocaml(Camlp4_config) = 80b5d58834366711574a5ec4dfb123fd   
   ocaml(Camlp4_import) = 4d17b58763ba1f0aac92fd5dbb558b59   
   ocaml(Char) = 3da72249626c7db769beafc97036cb4f
   ocaml(Complex) = 73899d718b62e5534e8737bb363dbf71
   ocaml(Filename) = 7cd172f02b7ee9b8d7bda3bb92144951
   ocaml(Format) = b7ba3152a5eec5609d6ab86e6c51eebb
   ocaml(Hashtbl) = ee2a3220e38a4350c5bc131ce9f3f6ce
   ocaml(Int32) = b2545c419b6b6a173cac4c0a3e7e0277
   ocaml(Int64) = d501d6e89fdce41c79f274fb464995d5
   ocaml(Lazy) = 4c7ed568fa7b5f73a2aa02eeb0e5e12b
   ocaml(Lexing) = 4d17267334f1a6c75730dc3fae21fb9b
   ocaml(List) = a0e2e49d266ff302f8667651a43f71ba
   ocaml(Nativeint) = 7233ce5207a538fea4f0c61ed411ea2c
   ocaml(Obj) = c827f726ce05da709cf7de58fc15e324
   ocaml(Parsing) = 29c3f123280f8e6e639cfb025b3c9a3f
   ocaml(Pa_type_conv) = 917c39ac24d30438f1e78e6e58840e45
   ocaml(Pervasives) = 88cb1505c8bdf9a4dcd2cdf3452732b4
   ocaml(Printf) = 807ecd3a1538992580464c03462c9964
   ocaml(Queue) = 56b5e04dcda600ae0cdf49a37f17fcd9
   ocaml(Set) = c4be5d24d30c129dd60d2739e54db7dd
   ocaml(Stream) = 91a43ea7fb16bf36f3f10c0dc7d08a0e
   ocaml(String) = ecc403546c1c50056801131811c39017
   ocaml(Sys) = 21bf525b2b3f3a46a54b96163adfe387
   ocaml(Unix) = 0596a58544f8cd88fed5bf5432a53d43
   ocaml(runtime) = 3.11.1

  ocaml-bin-prot-devel-1.2.20-1.fc12.x86_64.rpm
   ocaml-bin-prot-devel = 1.2.20-1.fc12
   ocaml-bin-prot-devel(x86-64) = 1.2.20-1.fc12
  =
   ocaml-bin-prot = 1.2.20-1.fc12

* %check is present and all tests pass:
   Ran: 67 tests in: 0.35 seconds.
   OK

   msgs: 100000  msg length: 461
   write time: 2.548s  write rate:  39244.14 msgs/s  write throughput: 17.25
MB/s
   read time: 2.619s   read rate:  38181.64 msgs/s   read throughput: 16.79
MB/s

* owns the directories it creates.
* doesn't own any directories it shouldn't.
X license files are duplicated.
* file permissions are appropriate.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %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.
? a .ml file is included
* .so files have no rpath

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