[Bug 500254] Pre-Review Request: jakarta-commons-jxpath - Simple XPath interpreter

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


Andrew Overholt <overholt at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |overholt at redhat.com
         AssignedTo|nobody at fedoraproject.org    |overholt at redhat.com
               Flag|                            |fedora-review?




--- Comment #1 from Andrew Overholt <overholt at redhat.com>  2009-05-14 15:54:17 EDT ---
It would be nice to have a URL for source0.  I found it upstream regardless,
matched the md5sum upstream to the one I downloaded and to the one in the SRPM.

I looked through a random sampling of 5 or so source files and they all say ASL
2.0 as does LICENSE.txt.  Therefore, the License field should reflect this.

There's a one line difference between the pom file included in the SRPM and the
one from upstream (line 41):

-      <organization/>
+      <organization></organization>

Otherwise, pre-review checklist seems okay to me.

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