Hi Andrey, Looking at http://openjdk.java.net/jtreg/vmoptions.html, I see we have both test.vm.opts and test.tool.vm.opts and the latter is supposed to take care of adding "-J".
+ VM_OPTIONS.stream().map(opt -> "-J" + opt).forEach(commands::add); + JAVA_OPTIONS.stream().map(opt -> "-J" + opt).forEach(commands::add); --- Maybe splitting on "\\s+" would be better. --- I think we should have test library methods to return the List<String> for java subprocesses, one that could try hard to get the option tokenization correct. --- On Wed, May 25, 2016 at 9:07 AM, Andrey Nazarov <andrey.x.naza...@oracle.com> wrote: > Some jar tests start VMs without passing vmoptions from jtreg. > > This fix pass jtreg's vmoptions and javaoptions to processes(java, jar, > javac) started by tests. > > webrev: http://cr.openjdk.java.net/~anazarov/8157850/webrev.01/ > > jbs: https://bugs.openjdk.java.net/browse/JDK-8157850 > > --Andrey >
