RFR: CODETOOLS-7902453: Jtreg JDK.getJDKVersion failed to load GetSystemProperty.class when debugging

[Patrick Zhang OS]

Hi Jon,
Thanks for your review.
Please see my updates: http://cr.openjdk.java.net/~qpzhang/codetools-7902453/webrev.02. Instead of modifying sysPropClassPath directly, I changed JarFinder.getPath() a little bit: if the URI is not JAR, try the root path of class files.
I would like to keep the fix for CLASSPATH, which should not be overwritten regardless. Maybe there were predefined values inside from users.

Regards
Patrick

-----Original Message-----
From: Jonathan Gibbons <jonathan.gibbons at oracle.com> 
Sent: Tuesday, April 23, 2019 10:50 PM
To: Patrick Zhang OS <patrick at os.amperecomputing.com>; code-tools-dev at openjdk.java.net
Subject: Re: RFR: CODETOOLS-7902453: Jtreg JDK.getJDKVersion failed to load GetSystemProperty.class when debugging

Since sysPropClassPath is supposed to contain the path, and does not, it would be better to fix sysPropClassPath to contain the correct value.

-- Jon

On 4/23/19 2:58 AM, Patrick Zhang OS wrote:
> Hi,
>
> I ran into an issue that jtreg reported that it cannot determine version for JDK when debugging, while the root cause is the root directory of class files could not be located correctly. There are a couple of methods to solve the problem (see details in JBS), the proposed one is to make com/sun/javatest/regtest/config/JDK.java not overwrite the CLASSPATH env var completely, update it with added paths instead.
> Please review, thanks.
>
> Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902453
> Webrev: 
> http://cr.openjdk.java.net/~qpzhang/codetools-7902453/webrev.01/
>
> Regards
> Patrick
>