request for review JDK-8211290

[Brent Christian]

Hi, Ivan

The changes look fine to me.
Please see that an automated test run that includes all the changed 
tests is performed.

Thanks,
-Brent

On 6/30/20 5:38 AM, Ivan Sipka wrote:
> Hi all,
> 
> kind reminder for RFR for JDK-8211974.
> 
> thank you,
> 
> ----- Original Message -----
> From: ivan.sipka at oracle.com
> To: core-libs-dev at openjdk.java.net
> Cc: igor.ignatyev at oracle.com
> Sent: Thursday, June 4, 2020 7:40:27 PM GMT +00:00 GMT Britain, Ireland, Portugal
> Subject: request for review JDK-8211290
> 
> Hi all,
> 
> please review the following changeset: http://cr.openjdk.java.net/~iignatyev/isipka/8211974/webrev.01/index.html
> for the JBS issue https://bugs.openjdk.java.net/browse/JDK-8211974
> 
> which moves the files:
> 
> open/test/jdk/lib/testlibrary/java/util/jar/Compiler.java
> open/test/jdk/lib/testlibrary/java/util/jar/JarBuilder.java
> 
> to files:
> 
> open/test/lib/jdk/test/lib/util/JarBuilder.java
> open/test/lib/jdk/test/lib/compiler/Compiler.java
> 
> and changes the relevant jtreg specification tags in dependent tests:
> 
> open/test/jdk/sun/net/www/protocol/jar/MultiReleaseJarURLConnection.java
> open/test/jdk/jdk/nio/zipfs/jarfs/JFSTester.java
> open/test/jdk/jdk/nio/zipfs/jarfs/MultiReleaseJarTest.java
> open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarAPI.java
> open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarHttpProperties.java
> open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarProperties.java
> open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarSecurity.java
> open/test/jdk/java/util/jar/JarFile/JarBacktickManifest.java
> open/test/jdk/java/net/URL/JarHandlerPkgPrefix/JarHandlerPkgPrefix.java
> 
> and adds relevant package imports to dependent file:
> 
> open/test/jdk/lib/testlibrary/java/util/jar/CreateMultiReleaseTestJars.java
> 
> jtreg command to run all dependent tests from repository root
> 
> jtreg -verbose:summary -ea -esa -a -conc:9 -jdk:build/linux-x64/jdk/ open/test/jdk/sun/net/www/protocol/jar/MultiReleaseJarURLConnection.java open/test/jdk/jdk/nio/zipfs/jarfs/JFSTester.java open/test/jdk/jdk/nio/zipfs/jarfs/MultiReleaseJarTest.java open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarAPI.java open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarHttpProperties.java open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarProperties.java open/test/jdk/java/util/jar/JarFile/mrjar/MultiReleaseJarSecurity.java open/test/jdk/java/util/jar/JarFile/JarBacktickManifest.java open/test/jdk/java/net/URL/JarHandlerPkgPrefix/JarHandlerPkgPrefix.java
> 
> thank you,
>