RFR(s): 8166624: java/util/jar/JarFile/mrjar regression tests has undeclared dependencies

Alan Bateman Alan.Bateman at oracle.com
Sat Sep 24 10:02:46 UTC 2016


On 23/09/2016 14:54, Sergei Kovalev wrote:

> Hello team,
>
> Could you please review a very small fix that relates to tests only.
>
> BugID: https://bugs.openjdk.java.net/browse/JDK-8166624
> Webrev: http://cr.openjdk.java.net/~skovalev/8166624/webrev.00/
"@modules java.compiler/javax.tools" looks odd, should this be simply 
"@modules java.compiler"?

In any case, I would have thought that the modules list in 
mrjar/TEST.properties was enough to ensure these tests don't run when 
the runtime under test doesn't have jdk.jartool or jdk.compiler. But 
perhaps the motivation of the change is to make it more fine grained?

-Alan


More information about the core-libs-dev mailing list