[ 
https://issues.apache.org/jira/browse/TIKA-756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13405078#comment-13405078
 ] 

Jukka Zitting commented on TIKA-756:
------------------------------------

OK, see my followup commits for the minor adjustments. There are a few things 
remaining:

* I didn't change the formatting of code inside tika-xmp/src to avoid breaking 
any pending changes you may have. It would be good however to unify the 
formatting of that code with the rest of Tika where we normally use four spaces 
(no tabs) for indentation and don't use a separate line for an opening brace.

* The copyright headers mention the IPTC Photo Metadata standard. I didn't 
notice specific IPTC metadata descriptions being included in the relevant 
source files, so can we drop that extra copyright notice?

* The tika-xmp component currently has a dependency on tika-parsers just to get 
the list of media types supported by relevant parser components. Could we 
rather make tika-parsers depend on tika-xmp and provide the {{Converter}} 
classes as parts of the relevant o.a.t.parser.* packages. The {{TikaToXMP}} 
class could access them using the same {{ServiceLoader}} mechanism as tika-core 
uses for {{Detector}} and {{Parser}} implementations?
                
> XMP output from Tika CLI
> ------------------------
>
>                 Key: TIKA-756
>                 URL: https://issues.apache.org/jira/browse/TIKA-756
>             Project: Tika
>          Issue Type: New Feature
>          Components: cli, metadata
>            Reporter: Jukka Zitting
>            Assignee: Jukka Zitting
>              Labels: metadata, xmp
>         Attachments: tika-xmp.patch
>
>
> It would be great if the Tika CLI could output metadata also in the XMP 
> format.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to