On 24/02/2016 04:01, Felix Yang wrote:
Jon and Alan,
     thanks a lot for this comment. Fixed it in new webrev:
http://cr.openjdk.java.net/~xiaofeya/8141609/webrev.01/

I think JrtfsJarTest will also need to check the `release` file in $JDK8_HOME to check that it's a JDK 8 release. Its a properties file so should be easy. I bring this up because I'm not sure that we can guarantee that JT_JAVA is a JDK 8 build.

On JrtfsTestMain then it looks like it's a copy of the jrtfs Basic.java test. I wonder if we should strip it down a bit to make it easier to maintain. I don't object to pushing the test as in into jake but there will be some clean-up required.

-Alan

Reply via email to