Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: constantine - Platform Constants for Java
https://bugzilla.redhat.com/show_bug.cgi?id=473530
Summary: Review Request: constantine - Platform Constants for Java Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: konrad@tylerc.org QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/constantine.spec SRPM URL: http://konradm.fedorapeople.org/fedora/SRPMS/constantine-0.1-1.fc9.src.rpm Description: Provides java values for common platform C constants (e.g. errno).
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=473530
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |473451
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=473530
--- Comment #1 from Conrad Meyer konrad@tylerc.org 2008-11-29 02:47:12 EDT --- New URLS: http://konradm.fedorapeople.org/fedora/SPECS/constantine.spec http://konradm.fedorapeople.org/fedora/SRPMS/constantine-0.4-1.fc9.src.rpm
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=473530
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |tibbs@math.uh.edu Flag| |fedora-review?
--- Comment #2 from Jason Tibbitts tibbs@math.uh.edu 2008-12-06 20:38:10 EDT --- It's been a while since I reviewed a java package. This one seems simple and clean enough. I note, though, that you don't include the gcj aot bits as http://fedoraproject.org/wiki/Packaging/GCJGuidelines suggests. Frankly I don't know enough about the Java situation to argue either way; on one hand, my understanding is that we still have platforms which benefit from native code generation, but on the other hand this package exists primarily for jruby which doesn't do the aot stuff either. Anyway, it isn't a blocker.
rpmlint says: constantine.noarch: W: no-documentation which normally would be OK, except that in this case at least the LICENSE file should be included.
Regarding the test suite, it would of course be good to run it, but if it doesn't work without hacks then I certainly can't fault you
So the only blocker I see is the inclusion of the license file.
* source files match upstream. sha256sum: f12bb1820fdcb30464929e605b5181c8ea95e9eea8294b86c00b15d7a9431c74 constantine-src-0.4.zip * 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. X license text not included in package. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * final provides and requires are sane: constantine = 0.4-1.fc11 = java jpackage-utils
* no duplicates in %files. * file permissions are appropriate. * code, not content. * %docs are not necessary for the proper functioning of the package.
Java-specific bits: * no pre-built jars * single jar, named after the package * jarfiles are under _javadir. * ant called properly.
The package review process needs reviewers! If you haven't done any package reviews recently, please consider doing one.
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=473530
--- Comment #3 from Conrad Meyer konrad@tylerc.org 2008-12-06 20:49:14 EDT --- New URLs:
http://konradm.fedorapeople.org/fedora/SPECS/constantine.spec http://konradm.fedorapeople.org/fedora/SRPMS/constantine-0.4-2.fc9.src.rpm
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=473530
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #4 from Jason Tibbitts tibbs@math.uh.edu 2008-12-07 13:27:01 EDT --- Looks good; thanks.
APPROVED
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=473530
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #5 from Conrad Meyer konrad@tylerc.org 2008-12-07 14:51:24 EDT --- New Package CVS Request ======================= Package Name: constantine Short Description: Platform Constants for Java Owners: konradm Branches: F-10 F-9 InitialCC:
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=473530
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #6 from Kevin Fenzi kevin@tummy.com 2008-12-07 19:29:54 EDT --- cvs done.
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=473530
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
--- Comment #7 from Conrad Meyer konrad@tylerc.org 2008-12-07 22:01:06 EDT --- http://koji.fedoraproject.org/koji/taskinfo?taskID=986525
Built in rawhide, closing.
package-review@lists.fedoraproject.org