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@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |tibbs@xxxxxxxxxxx Flag| |fedora-review? --- Comment #3 from Jason Tibbitts <tibbs@xxxxxxxxxxx> 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review