request for review JDK-8211290

[Roger Riggs]

Hi Ivan,

In test/jdk/java/util/jar/JarFile/JarBacktickManifest.java, I don't 
think you want the new blank line
after the removed @summary.

Otherwise, looks fine.

Roger


On 6/30/20 5:15 PM, Brent Christian wrote:
> 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,
>>