RFR(XS) : 8202559 : Tests which start VM using JNI start failing after compile upgrade to VC 2017

[Erik Joelsson]

Looks good to me.

/Erik


On 2018-06-18 16:19, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8202559/webrev.00/index.html
>> 12 lines changed: 5 ins; 0 del; 7 mod
> Hi all,
>
> could you please review this small fix for ExecDriver class to add <jdk>/bin directory to PATH in 'launcher' mode on windows?
>
> we started to observer that the tests which use ExecDriver's launcher mode fail due to missed a shared library. the reason is that ExecDriver's adds only <jdk>/bin/<variant>, but has to add both <jdk>/bin and <jdk>/bin/<variant>
>
> webrev: http://cr.openjdk.java.net/~iignatyev//8202559/webrev.00/index.html
> JBS: https://bugs.openjdk.java.net/browse/JDK-8202559
> testing: affected tests (none in open) on windows and non-windows platform
>
> Thanks,
> -- Igor