You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "Tyler Palsulich (JIRA)" <ji...@apache.org> on 2015/03/30 15:20:52 UTC

[jira] [Commented] (TIKA-1587) ForkParser::setJavaCommand should take List

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

Tyler Palsulich commented on TIKA-1587:
---------------------------------------

Thank you for reporting this! It seems like a definite problem. Is there any way you can provide a patch?

> ForkParser::setJavaCommand should take List<String>
> ---------------------------------------------------
>
>                 Key: TIKA-1587
>                 URL: https://issues.apache.org/jira/browse/TIKA-1587
>             Project: Tika
>          Issue Type: Improvement
>          Components: parser
>    Affects Versions: 1.7
>            Reporter: Oleg Oshmyan
>
> ForkParser::setJavaCommand currently takes a string and splits it on whitespace. This makes it impossible to use commands with paths that contain spaces. In particular, it makes it impossible to reliably use System.getProperty("java.home") in order to launch the same Java that the current process is running in, because it might contain spaces. If it would just take a List<String> and pass (a clone of) it directly to ProcessBuilder, this wouldn't be a problem.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)