Bug 518233

Summary:	Review Request: jakarta-commons-jxpath - Simple XPath interpreter
Product:	[Fedora] Fedora	Reporter:	Alexander Kurtakov <akurtako>
Component:	Package Review	Assignee:	Andrew Overholt <overholt>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting, overholt
Target Milestone:	---	Flags:	overholt: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2009-08-20 13:54:46 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	429551

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.
http://koji.fedoraproject.org/koji/buildinfo?buildID=128061