[8u] RFR 8229501: jdk/test/lib/testlibrary/ClassFileInstaller.java should match hotspot//test/testlibrary version

[Verghese, Clive]

Hi Zhengyu, 

Thank you for reviewing the change. 

Regards,
Clive Verghese

On 8/14/19, 10:40 AM, "Hohensee, Paul" <hohensee at amazon.com> wrote:

    Thanks, Zhengyu. I've tagged the JBS issue with jdk8u-fix-request, will push on Clive's behalf when I see jdk8u-fix-yes.
    
    Paul
    
    On 8/14/19, 7:08 AM, "jdk8u-dev on behalf of Zhengyu Gu" <jdk8u-dev-bounces at openjdk.java.net on behalf of zgu at redhat.com> wrote:
    
        Hi Clive,
        
        Looks good to me. I think you need permission to push.
        
        Thanks,
        
        -Zhengyu
        
        On 8/13/19 6:12 PM, Verghese, Clive wrote:
        > Hi,
        > 
        > Requesting review for webrev : http://cr.openjdk.java.net/~phh/8229501/webrev.8u.00/
        > JBS Issue : https://bugs.openjdk.java.net/browse/JDK-8229501
        > Email discussion : https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-August/010029.html
        > 
        > hotspot/test/testlibrary/ClassFileInstaller.java and jdk/test/lib/testlibrary/ClassFileInstaller were merged as part of the JDK10 repo merge. For 8u, we should sync the JDK version with the Hotspot version in order to facilitate backports of patches such as the one for JDK-8216401<https://bugs.openjdk.java.net/browse/JDK-8216401> that assume the merged version.
        > 
        > Testing
        >                  Validated that the test depending on ClassFileInstaller pass.
        > 
        > 
        > Regards,
        > Clive Verghese
        >