Bug 518233 - Review Request: jakarta-commons-jxpath - Simple XPath interpreter
Summary: Review Request: jakarta-commons-jxpath - Simple XPath interpreter
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Andrew Overholt
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 429551
TreeView+ depends on / blocked
Reported: 2009-08-19 15:45 UTC by Alexander Kurtakov
Modified: 2009-08-20 13:54 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-08-20 13:54:46 UTC
Type: ---
overholt: fedora-review+

Attachments (Terms of Use)

Description Alexander Kurtakov 2009-08-19 15:45:38 UTC
Spec URL: http://akurtakov.fedorapeople.org/jakarta-commons-jxpath.spec
SRPM URL: http://akurtakov.fedorapeople.org/jakarta-commons-jxpath-1.2-7.fc12.src.rpm
Description: Defines a simple interpreter of an expression language called XPath.
JXPath applies  XPath  expressions to graphs of objects of all kinds:
JavaBeans, Maps, Servlet contexts, DOM etc, including mixtures thereof.

Comment 1 Alexander Kurtakov 2009-08-19 15:47:09 UTC
This is a full review request.  The pre-review request was bug #500254.

Comment 2 Andrew Overholt 2009-08-19 16:05:02 UTC
- extra spaces interspersed in %description
- md5sums are different but contents are the same
- svn revision I got was different # ... perhaps we should just remove that comment
- licensing, naming fine
- locations fine
- rpmlint clean (except for maven fragment in /etc)
- we should go to 1.3 ASAP but I can understand the strict need for 1.2 to get maven updated

Please fix those minor things.  Review approved since they're not blockers.

Comment 3 Alexander Kurtakov 2009-08-20 05:53:21 UTC
Should we do something else here because the package is already in cvs?
Cvs was created and package imported from the Pre-Review bug.
Maybe we should just build in rawhide?

Comment 4 Alexander Kurtakov 2009-08-20 13:54:46 UTC
Build in rawhide.

Note You need to log in before you can comment on or make changes to this bug.