From jonathan.gibbons at oracle.com  Wed Sep 13 22:07:48 2017
From: jonathan.gibbons at oracle.com (Jonathan Gibbons)
Date: Wed, 13 Sep 2017 15:07:48 -0700
Subject: RFR 7902028: Make com.sun.javatest.regtest.config.TestProperties
 API public
In-Reply-To: <38C40326-AABF-4154-9E8C-A800C411CBF9@oracle.com>
References: <9EE49B42-1895-4848-AD22-8FFB7C00EA41@oracle.com>
 <38C40326-AABF-4154-9E8C-A800C411CBF9@oracle.com>
Message-ID: <59B9AC34.9000007@oracle.com>

Is this OK?

-- Jon

$ hg diff -c tip
diff -r 815bc88d8313 -r 297e70583720 
src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java
--- 
a/src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java 
Mon Jul 31 13:27:30 2017 -0700
+++ 
b/src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java 
Wed Sep 13 15:06:07 2017 -0700
@@ -208,6 +208,10 @@
          return properties.getExternalLibs(td.getFile());
      }

+    public Set<String> getDefaultModules(TestDescription td) throws 
TestSuite.Fault {
+        return properties.getModules(td.getFile());
+    }
+
      public ExtraPropDefns getExtraPropDefns() {
          return properties.getExtraPropDefns();
      }



On 09/12/2017 04:56 PM, Alexandre (Shura) Iline wrote:
> I realized I have forgotten to CC JTReg dev alias.
>
>> On Sep 11, 2017, at 10:44 AM, Alexandre (Shura) Iline <alexandre.iline at oracle.com> wrote:
>>
>> Jon,
>>
>> Let me know if you would be OK with this fix. I, for now, only need the modules, but I through it would be somewhat weird to only make that public.
>>
>> Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902028
>> Webrev: http://cr.openjdk.java.net/~shurailine/7902028/webrev.00/
>>
>> Thank you.
>>
>> Shura
>>


From jonathan.gibbons at oracle.com  Wed Sep 13 23:06:42 2017
From: jonathan.gibbons at oracle.com (jonathan.gibbons at oracle.com)
Date: Wed, 13 Sep 2017 23:06:42 +0000
Subject: hg: code-tools/jtreg: update makefile names for JDK 9
Message-ID: <201709132306.v8DN6gEl018760@aojmv0008.oracle.com>

Changeset: d94105d93b14
Author:    jjg
Date:      2017-09-13 16:03 -0700
URL:       http://hg.openjdk.java.net/code-tools/jtreg/rev/d94105d93b14

update makefile names for JDK 9

! make/Defs.gmk
! test/addmods/AddModulesTest.gmk
! test/jdkModulesTest/JDKModulesTest.gmk
! test/modlibs/ModLibsTest.gmk
! test/moduleOpens/ModuleOpensTest.gmk
! test/modules/ModulesTest.gmk


From jonathan.gibbons at oracle.com  Wed Sep 13 23:39:14 2017
From: jonathan.gibbons at oracle.com (jonathan.gibbons at oracle.com)
Date: Wed, 13 Sep 2017 23:39:14 +0000
Subject: hg: code-tools/jtreg: 2 new changesets
Message-ID: <201709132339.v8DNdEdp001607@aojmv0008.oracle.com>

Changeset: f765dc0df79c
Author:    jjg
Date:      2017-09-13 16:04 -0700
URL:       http://hg.openjdk.java.net/code-tools/jtreg/rev/f765dc0df79c

expose default modules for test

! src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java

Changeset: 765927ec595d
Author:    jjg
Date:      2017-09-13 16:36 -0700
URL:       http://hg.openjdk.java.net/code-tools/jtreg/rev/765927ec595d

7901983: jtreg should update LD_LIBRARY_PATH only for actions with /native option

! src/share/classes/com/sun/javatest/regtest/exec/Action.java
! src/share/classes/com/sun/javatest/regtest/exec/CompileAction.java
! src/share/classes/com/sun/javatest/regtest/exec/MainAction.java
! src/share/classes/com/sun/javatest/regtest/exec/RegressionScript.java
! src/share/classes/com/sun/javatest/regtest/exec/ShellAction.java
! src/share/classes/com/sun/javatest/regtest/tool/Tool.java


From jonathan.gibbons at oracle.com  Thu Sep 14 00:20:55 2017
From: jonathan.gibbons at oracle.com (jonathan.gibbons at oracle.com)
Date: Thu, 14 Sep 2017 00:20:55 +0000
Subject: hg: code-tools/jtreg: 7902014: jtreg TimeoutHandler should use
 renamed Process.pid method
Message-ID: <201709140020.v8E0Ktum020441@aojmv0008.oracle.com>

Changeset: d72212487e94
Author:    jjg
Date:      2017-09-13 17:17 -0700
URL:       http://hg.openjdk.java.net/code-tools/jtreg/rev/d72212487e94

7902014: jtreg TimeoutHandler should use renamed Process.pid method

! src/share/classes/com/sun/javatest/regtest/TimeoutHandler.java