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@xxxxxxxxxxx> 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review