Spec URL: http://mef.fedorapeople.org/packages/java-libraries/jempbox.spec SRPM URL: http://mef.fedorapeople.org/packages/java-libraries/jempbox-0.2.0-2.src.rpm Description: JempBox is an open source Java library that implements Adobe's XMP specification. Its features include: - Reading/Writing XMP streams - High level API to standard XMP schemas (NB: I don't really know what XMP is, but this is an indirect dependency of another Java package I want to get in ...)
Updated with a better BuildRoot and with dist in the version: http://mef.fedorapeople.org/packages/java-libraries/jempbox.spec http://mef.fedorapeople.org/packages/java-libraries/jempbox-0.2.0-3.fc10.src.rpm
Created attachment 327246 [details] [Review] Attached is my initial review. Thanks for submitting this software to Fedora! General comments: - Why the %update_maven_depmap and friends? It uses ant to build, not maven (note: I don't think this is wrong necessarily, I'm just unfamiliar with maven and would like to understand if this should be part of the java packaging guidelines for ant-built packages as well as maven). Will approve as soon as "BAD" marked thing is either fixed or explained.
XMP is adobe's new proprietary format that implements metadata in files, kinda like exif, but supposedly somehow better. It is a patented "technology", however Adobe have provided a "public patent licence" http://www.adobe.com/devnet/xmp/pdfs/xmp_public_patent_license.pdf It seems OK to me, but IANAL -- are there any legal issues to consider? The clause "A claim is necessarily infringed only when there is no reasonable way to avoid infringement of that claim when defining, creating or processing metadata." seems a bit nebulous.
We should have RH Legal look at it just in case. Blocking FE-Legal for now.
Everything checks out here. Lifting FE-Legal.
I'm working on packaging up jempbox 0.8.0 from Apache Incubating: http://incubator.apache.org/pdfbox/ Are you still wanting to proceed with this review or shall I close and submit my own?
(In reply to comment #6) > I'm working on packaging up jempbox 0.8.0 from Apache Incubating: > > http://incubator.apache.org/pdfbox/ > > Are you still wanting to proceed with this review or shall I close and submit > my own? You can go ahead and submit your package -- sorry I let this sit for so long. :(
*** This bug has been marked as a duplicate of bug 527996 ***