[8u] RFR 8216401: Allow "file:" URLs in Class-Path of local JARs
    Verghese, Clive 
    verghese at amazon.com
       
    Tue Aug 13 18:59:45 UTC 2019
    
    
  
Hi Zhengyu, 
Instead of importing the writeToDisk function to the test, We could update the ClassFileInstaller to be similar to the one in tip. [1]
http://cr.openjdk.java.net/~phh/8216401/webrev.8u0.00/
Regards,
Clive Verghese
[1] : https://hg.openjdk.java.net/jdk/jdk/file/be8c11fc16bb/test/lib/ClassFileInstaller.java
On 8/12/19, 11:38 AM, "jdk8u-dev on behalf of Zhengyu Gu" <jdk8u-dev-bounces at openjdk.java.net on behalf of zgu at redhat.com> wrote:
    Please review 8u backport of above CR.
    JDK13 patch does not apply cleanly. However, the fix itself applies 
    cleanly, just not the test.
    
    8u ClassFileInstaller does not have some APIs that exist in JDK13. 
    Therefore, I copied/pasted corresponding code from JDK13's 
    ClassFileInstaller into the test (writeClassToDisk and writeToDisk).
    
    Original bug: https://bugs.openjdk.java.net/browse/JDK-8216401
    Original Webrev: 
    https://mail.openjdk.java.net/pipermail/core-libs-dev/2019-January/057868.html
    Original review thread: 
    https://mail.openjdk.java.net/pipermail/core-libs-dev/2019-January/057868.html
    
    8u webrev: http://cr.openjdk.java.net/~zgu/JDK-8216401-8u/webrev.00/
    
    Test:
       Run attached test case (before and after fix) on Linux 86_64.
    
    Thanks,
    
    -Zhengyu
    
    
    
More information about the jdk8u-dev
mailing list