[ 
https://issues.apache.org/jira/browse/EXEC-38?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12702894#action_12702894
 ] 

Sebb commented on EXEC-38:
--------------------------

Fixed in SVN trunk:

URL: http://svn.apache.org/viewvc?rev=768717&view=rev
Log:
EXEC-38 - use addArgument() not addArguments()
Also remove Jakarta branding
Add another substitution example

> Tutorial example uses incorrect API call.
> -----------------------------------------
>
>                 Key: EXEC-38
>                 URL: https://issues.apache.org/jira/browse/EXEC-38
>             Project: Commons Exec
>          Issue Type: Bug
>    Affects Versions: 1.0.0
>            Reporter: Peter Henderson
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> http://commons.apache.org/exec/tutorial.html
> The section "Build the Command Line Incrementally" uses 
> commandLine.addArguments(file.getAbsolutePath());
> which will go horribly wrong if the path has spaces in it.
> It should be using
> commandLine.addArgument(file.getAbsolutePath());
> Note the singular addArgument
> The two preceding calls to addArgument could also benefit from being changed.
> Peter

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to