On Tue, 24 Aug 2021 19:06:55 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> test/jdk/java/lang/ProcessBuilder/Basic.java line 30:
>> 
>>> 28:  *      6464154 6523983 6206031 4960438 6631352 6631966 6850957 6850958
>>> 29:  *      4947220 7018606 7034570 4244896 5049299 8003488 8054494 8058464
>>> 30:  *      8067796 8224905 8263729 8265173 8272600 8231297
>> 
>> The test should also be modified to use `@run 
>> main/othervm/native/timeout=300` so that this test will be flagged by jtreg 
>> if `-nativepath:` is not specified.
>
> It should be possible to run this test as a main, without the overhead of 
> building the native image.
> The use of a Java child greatly reduces the complexity of the test and 
> improves its maintainability.
> Requiring a native special built program raises the overhead considerably.
> And all because the VM can't or won't allow its output to be managed.

In the same way the test uses:

` private static final String[] winEnvCommand = {"cmd.exe", "/c", "set"};`

you could also have:

` private static final String[] winSleepCommand = {"cmd.exe", "/c", "timeout", 
"/T", "60", "/NOBREAK"};`

-------------

PR: https://git.openjdk.java.net/jdk/pull/5239

Reply via email to