[
https://issues.apache.org/jira/browse/TIKA-1706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14698401#comment-14698401
]
Hudson commented on TIKA-1706:
------------------------------
SUCCESS: Integrated in tika-trunk-jdk1.7 #826 (See
[https://builds.apache.org/job/tika-trunk-jdk1.7/826/])
Use a consistent version of Commons IO everywhere, enable the Forbidden APIs
check for it, and fix problems it found TIKA-1706 (nick:
http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1696079)
* /tika/trunk/tika-app/pom.xml
* /tika/trunk/tika-batch/pom.xml
* /tika/trunk/tika-example/pom.xml
*
/tika/trunk/tika-example/src/main/java/org/apache/tika/example/DirListParser.java
*
/tika/trunk/tika-example/src/main/java/org/apache/tika/example/MyFirstTika.java
*
/tika/trunk/tika-example/src/main/java/org/apache/tika/example/RollbackSoftware.java
*
/tika/trunk/tika-example/src/test/java/org/apache/tika/example/SimpleTextExtractorTest.java
* /tika/trunk/tika-parent/pom.xml
* /tika/trunk/tika-parsers/pom.xml
* /tika/trunk/tika-server/pom.xml
*
/tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TranslateResource.java
> Bring back commons-io to tika-core
> ----------------------------------
>
> Key: TIKA-1706
> URL: https://issues.apache.org/jira/browse/TIKA-1706
> Project: Tika
> Issue Type: Improvement
> Components: core
> Reporter: Yaniv Kunda
> Priority: Minor
> Fix For: 1.11
>
> Attachments: TIKA-1706.patch
>
>
> TIKA-249 inlined select commons-io classes in order to simplify the
> dependency tree and save some space.
> I believe these arguments are weaker nowadays due to the following concerns:
> - Most of the non-core modules already use commons-io, and since tika-core is
> usually not used by itself, commons-io is already included with it
> - Since some modules use both tika-core and commons-io, it's not clear which
> code should be used
> - Having the inlined classes causes more maintenance and/or technology debt
> (which in turn causes more maintenance)
> - Newer commons-io code utilizes newer platform code, e.g. using Charset
> objects instead of encoding names, being able to use StringBuilder instead of
> StringBuffer, and so on.
> I'll be happy to provide a patch to replace usages of the inlined classes
> with commons-io classes if this is accepted.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
