Uploaded image for project: 'Tika'
  1. Tika
  2. TIKA-1587

ForkParser::setJavaCommand should take List<String>

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.7
    • 1.8
    • parser
    • None

    Description

      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.

      Attachments

        1. TIKA-1587.patch
          3 kB
          Oleg Oshmyan

        Activity

          People

            grossws Konstantin Gribov
            astiob Oleg Oshmyan
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: