> On May 9, 2017, at 4:23 PM, Brent Christian <brent.christ...@oracle.com>
> 
> I've removed the test case for automatic modules, and added a @run action for 
> each policy file + system classloader configuration.  I also removed the code 
> to compile test sources, using @build instead.
> 
> I also made some (hopefully) clarifying changes to the code.
> 
> The String[] argument to execute() is now separated into args/status/message, 
> instead of being lumped into one String[], and then having to be split() out 
> of a single String.  Where execute() is called, it's easier to read the args 
> in order (versus assembled using String.format()).
> 
> I also simplified the code for expected status/output, and made some path 
> variable names more descriptive.
> 
> Webrev:
> http://cr.openjdk.java.net/~bchristi/8177328/webrev.01/

Thanks for the clean up.  Does each @run action need 4 mins timeout?  Can it 
restore to the default timeout?

Mandy

Reply via email to