You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Commit Tag Bot (JIRA)" <ji...@apache.org> on 2013/03/22 17:39:15 UTC

[jira] [Commented] (SOLR-3848) dataimporthandler-extras depends on Tika but doesn't have it in it's ivy deps

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

Commit Tag Bot commented on SOLR-3848:
--------------------------------------

[branch_4x commit] Chris M. Hostetter
http://svn.apache.org/viewvc?view=revision&revision=1386792

SOLR-3848: comments explaining workarround (merge r1386790)

                
> dataimporthandler-extras depends on Tika but doesn't have it in it's ivy deps
> -----------------------------------------------------------------------------
>
>                 Key: SOLR-3848
>                 URL: https://issues.apache.org/jira/browse/SOLR-3848
>             Project: Solr
>          Issue Type: Bug
>          Components: contrib - DataImportHandler
>            Reporter: Hoss Man
>
> Noticed this while dealing with SOLR-3759...
> * solr/contrib/dataimporthandler-extras contains MailEntityProcessor and TikaEntityProcessor
> * both of these classes have acompile & runtime dependency on org.apache.tika.\*
> * solr/contrib/dataimporthandler-extras/ivy.xml does not mention any external dependencies
> * solr/contrib/dataimporthandler-extras/build.xml has a "resolve-extraction-libs" to force solr/contrib/extraction to fetch it's deps so that dataimporthandler-extras can use them directly
> * solrconfig.xml files in example-DIH point to the contrib/extraction/lib/ dir to get the Tika dependencies for demo purposes
> I believe this is all intentional so that we don't have two copies of all the tika jars floating around, particularly in the binary releases, but even though i'm one of the people who was involved in setting things up this way in dataimporthandler-extras/build.xml, it still confused/surprised me...
> https://svn.apache.org/viewvc?view=revision&revision=1307563
> https://svn.apache.org/viewvc?view=revision&revision=1309503
> I think at a minimum, we should probably add some comments to dataimporthandler-extras/ivy.xml about this kludge, and probably call it out more in the various example-DIH/\*/solrconfig.xml files as well.  That said: If anyone feels strongly that we should "fix" this so that dataimporthandler-extras/ivy.xml explicitly fetches the tika deps - please speak up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org