Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: jempbox - A Java library that implements Adobe's XMP specification
https://bugzilla.redhat.com/show_bug.cgi?id=472792
Summary: Review Request: jempbox - A Java library that implements Adobe's XMP specification Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mefoster@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
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 ...)
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=472792
--- Comment #1 from Mary Ellen Foster mefoster@gmail.com 2008-12-09 11:36:56 EDT --- 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...
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=472792
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |konrad@tylerc.org AssignedTo|nobody@fedoraproject.org |konrad@tylerc.org Flag| |fedora-review?
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=472792
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
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=472792
--- Comment #2 from Conrad Meyer konrad@tylerc.org 2008-12-17 07:49:57 EDT --- Created an attachment (id=327246) --> (https://bugzilla.redhat.com/attachment.cgi?id=327246) [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.
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=472792
D Haley mycae@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mycae@yahoo.com
--- Comment #3 from D Haley mycae@yahoo.com 2008-12-28 08:28:06 EDT --- 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.
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=472792
Dominik 'Rathann' Mierzejewski rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |182235
--- Comment #4 from Dominik 'Rathann' Mierzejewski rpm@greysector.net 2009-01-19 18:16:31 EDT --- We should have RH Legal look at it just in case. Blocking FE-Legal for now.
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=472792
Tom "spot" Callaway tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tcallawa@redhat.com Blocks|182235 |
--- Comment #5 from Tom "spot" Callaway tcallawa@redhat.com 2009-02-12 14:19:12 EDT --- Everything checks out here. Lifting FE-Legal.
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=472792
Conrad Meyer konrad@tylerc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(mefoster@gmail.co | |m)
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=472792
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |orion@cora.nwra.com Flag| |needinfo?(mefoster@gmail.co | |m)
--- Comment #6 from Orion Poplawski orion@cora.nwra.com 2009-10-05 19:07:39 EDT --- 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?
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=472792
Mary Ellen Foster mefoster@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(mefoster@gmail.co | |m), | |needinfo?(mefoster@gmail.co | |m) |
--- Comment #7 from Mary Ellen Foster mefoster@gmail.com 2009-10-08 06:28:43 EDT --- (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. :(
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=472792
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |DUPLICATE
--- Comment #8 from Orion Poplawski orion@cora.nwra.com 2009-10-08 10:45:18 EDT ---
*** This bug has been marked as a duplicate of 527996 ***
package-review@lists.fedoraproject.org