# HG changeset patch # User iignatyev # Date 1397746449 -14400 # Node ID 3cedd541a6f37ddfe86600ae0b1d1eb7a9576f4e # Parent c55dabab1599b6b3b60d0f7e65737d3b4ace3910 8039260: c.o.j.t.ProcessTools::createJavaProcessBuilder(boolean, String... ) must also take TestJavaOptions Reviewed-by: kvn, iignatyev Contributed-by: lev.priima@oracle.com diff -r c55dabab1599 -r 3cedd541a6f3 hotspot/test/testlibrary/com/oracle/java/testlibrary/ProcessTools.java --- a/hotspot/test/testlibrary/com/oracle/java/testlibrary/ProcessTools.java Wed Apr 16 14:49:03 2014 -0700 +++ b/hotspot/test/testlibrary/com/oracle/java/testlibrary/ProcessTools.java Thu Apr 17 18:54:09 2014 +0400 @@ -145,18 +145,15 @@ return createJavaProcessBuilder(false, command); } - public static ProcessBuilder createJavaProcessBuilder(boolean addTestVmOptions, String... command) throws Exception { + public static ProcessBuilder createJavaProcessBuilder(boolean addTestVmAndJavaOptions, String... command) throws Exception { String javapath = JDKToolFinder.getJDKTool("java"); ArrayList args = new ArrayList<>(); args.add(javapath); Collections.addAll(args, getPlatformSpecificVMArgs()); - if (addTestVmOptions) { - String vmopts = System.getProperty("test.vm.opts"); - if (vmopts != null && vmopts.length() > 0) { - Collections.addAll(args, vmopts.split("\\s")); - } + if (addTestVmAndJavaOptions) { + Collections.addAll(args, Utils.getTestJavaOpts()); } Collections.addAll(args, command);