Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Using Quotes within getRuntime().exec

I'd like to invoke bash using a string as input. Something like:

sh -l -c "./foo"

I'd like to do this from Java. Unfortunately, when I try to invoke the command using getRuntime().exec, I get the following error:

      foo": -c: line 0: unexpected EOF while looking for matching `"'

      foo": -c: line 1: syntax error: unexpected end of file

It seems to be related to my string not being terminated with an EOF.

Is there a way to insert a platform specific EOF into a Java string? Or should I be looking for another approach, like writing to a temp script before invoking "sh" ?

like image 737
Daniel Avatar asked Oct 02 '08 11:10

Daniel


3 Answers

Use this:

Runtime.getRuntime().exec(new String[] {"sh", "-l", "-c", "./foo"});

Main point: don't put the double quotes in. That's only used when writing a command-line in the shell!

e.g., echo "Hello, world!" (as typed in the shell) gets translated to:

Runtime.getRuntime().exec(new String[] {"echo", "Hello, world!"});

(Just forget for the moment that the shell normally has a builtin for echo, and is calling /bin/echo instead. :-))

like image 125
Chris Jester-Young Avatar answered Nov 19 '22 19:11

Chris Jester-Young


Windows command lines behave differently from UNIX, Mac OS X and GNU/Linux.

On Windows the process receives the input text verbatim after the executable name (and space). It's then up to the program to parse the command line (which is usually done implicitly, the programmer is often clueless about the process).

In GNU/Linux the shell processes the command line, guaranteeing the familiar array of strings passed to C's main function. You don't have that shell. The best approach (even on Windows) is to use one of the form of exec where you pass each command line argument individually in its own String.

Process exec​(String[] cmdarray)    
Process exec​(String[] cmdarray, String[] envp)     
Process exec​(String[] cmdarray, String[] envp, File dir)

Or better, java.lang.ProcessBuilder.

You can get a shell to do the parsing for you if you really want. This would make your example look something like (untested):

Runtime.getRuntime().exec(new String[] {
    "sh", "-c", "sh -l -c \"echo foo; echo bar;\""
});
like image 40
Tom Hawtin - tackline Avatar answered Nov 19 '22 18:11

Tom Hawtin - tackline


EOF is NOT a character, so there's no way to write an EOF. You've forgotten to close a quoted string.

like image 4
Lev Avatar answered Nov 19 '22 19:11

Lev