RFR (S) 8007142: Add utility classes for writing better multiprocess tests in jtreg (original) (raw)
Yekaterina Kantserova yekaterina.kantserova at oracle.com
Wed Feb 6 15:40:23 UTC 2013
- Previous message: RFR (S) 8007142: Add utility classes for writing better multiprocess tests in jtreg
- Next message: RFR: race with nested repos in hgforest.sh
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Alan,
thanks for the review!
// Katja
On 02/05/2013 12:54 PM, Alan Bateman wrote:
On 05/02/2013 10:51, Yekaterina Kantserova wrote:
On 02/04/2013 02:18 PM, Alan Bateman wrote:
On 04/02/2013 11:03, Yekaterina Kantserova wrote:
But JDKToolFinder.getJDKTool is used even for finding java, e.g. I need to fork a java process - JDKToolFinder.getJDKTool("java") for some JRE tests? It might be better to rename JDKToolFinder to something like JdkFinder and have it define getJavaLauncher() and getTool(String tool) methods. I think it's good idea to separate java launcher and tools. Here is the new JdkFinder: http://cr.openjdk.java.net/~ykantser/8007142/webrev.02/test/lib/testlibrary/jdk/testlibrary/JdkFinder.java.html. It now possible to retrieve java/tools both from the test.jdk and compile.jdk. If you don't agree then you could at least change ProcessTools.createJavaProcessBuilder so that it looks like "java" in the JDK/JRE under test. JDKToolFinder.getJDKTool("java") became JdkFinder.getJavaLauncher in the createJavaProcessBuilder (http://cr.openjdk.java.net/~ykantser/8007142/webrev.02/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java.html). -Alan. It looks right to me now. -Alan
- Previous message: RFR (S) 8007142: Add utility classes for writing better multiprocess tests in jtreg
- Next message: RFR: race with nested repos in hgforest.sh
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]