[Bug 473530] New: Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail], [e-mail]
   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).

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |473451

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review?

--- Comment #2 from Jason Tibbitts   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.

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #4 from Jason Tibbitts   2008-12-07 13:27:01 EDT ---
Looks good; thanks.

APPROVED

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

--- Comment #5 from Conrad Meyer   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:

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

--- Comment #6 from Kevin Fenzi   2008-12-07 19:29:54 EDT ---
cvs done.

[Bug 473530] Review Request: constantine - Platform Constants for Java

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE

--- Comment #7 from Conrad Meyer   2008-12-07 22:01:06 EDT ---
http://koji.fedoraproject.org/koji/taskinfo?taskID=986525

Built in rawhide, closing.