Re: RFR: JDK-8319569: Several java/util tests should be updated to accept VM flags [v3]

2023-11-21 Thread Lance Andersen
On Mon, 20 Nov 2023 19:15:18 GMT, Justin Lu wrote: >> Please review this PR which allows these _j.util_ tests to launch new JVM >> processes with VM flags, >> >> This is primarily done using by switching to >> `ProcessTools::createTestJavaProcessBuilder`. > > Justin Lu has updated the pull

Re: RFR: JDK-8319569: Several java/util tests should be updated to accept VM flags [v3]

2023-11-20 Thread Naoto Sato
On Mon, 20 Nov 2023 19:15:18 GMT, Justin Lu wrote: >> Please review this PR which allows these _j.util_ tests to launch new JVM >> processes with VM flags, >> >> This is primarily done using by switching to >> `ProcessTools::createTestJavaProcessBuilder`. > > Justin Lu has updated the pull

Re: RFR: JDK-8319569: Several java/util tests should be updated to accept VM flags [v3]

2023-11-20 Thread Justin Lu
> Please review this PR which allows these _j.util_ tests to launch new JVM > processes with VM flags, > > This is primarily done using by switching to > `ProcessTools::createTestJavaProcessBuilder`. Justin Lu has updated the pull request incrementally with one additional commit since the

Re: RFR: JDK-8319569: Several java/util tests should be updated to accept VM flags [v3]

2023-11-20 Thread Justin Lu
On Fri, 17 Nov 2023 20:17:28 GMT, Justin Lu wrote: >> test/jdk/java/util/Currency/PropertiesTest.sh line 30: >> >>> 28: # @summary tests the capability of replacing the currency data with user >>> 29: # specified currency properties file >>> 30: # @requires vm.flagless >> >> Does this