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

[Jonathan Gibbons]

> I would like to keep the fix for CLASSPATH, which should not be overwritten regardless. Maybe there were predefined values inside from users.

That would be inappropriate, and against long-standing jtreg policy of 
being susceptible to the user's environment.

-- Jon


On 04/24/2019 12:46 AM, Patrick Zhang OS wrote:
> 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
>>