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

Robert Muir commented on SOLR-3848:
-----------------------------------

yeah that fix might be more technically correct, but could also be more 
confusing. we would have to see what it looks like.

Maybe better to just have a build.xml thats confusing than a build.xml AND 
ivy.xml thats confusing.
                
> 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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to