Review request: JDK-8012453 (process) Runtime.exec(String) fails if command contains spaces [win]
Alan Bateman
Alan.Bateman at oracle.com
Fri Apr 26 12:24:26 UTC 2013
On 25/04/2013 15:41, Alexey Utkin wrote:
> Alan,
>
> Thanks for your comments.
> I did the changes in code.
> Please, read my answers below.
>
> Here is fixed version:
> http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-8012453/webrev.02/
The implementation changes look good now.
> :
>
>> One thing I see is that it doesn't read from the child's output/error
>> streams so if there is a problem then it will be hard to diagnose
>> from the logs.
> The diagnostic includes logging of the command line in problem. It
> should not be a problem to localize issue.
We'll see, sometimes it is hard to diagnose issues when you can see the
output from the child. What you have is fine and we can easily change
the test if we run into issues.
-Alan
More information about the core-libs-dev
mailing list