RFR: 8339460: CDS error when module is located in a directory with space in the name [v2]

Maxim Kartashev mkartashev at openjdk.org
Tue Sep 24 06:20:55 UTC 2024


On Mon, 23 Sep 2024 22:53:22 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> Maxim Kartashev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improved the test and excluded it from hotspot_appcds_dynamic
>
> test/hotspot/jtreg/runtime/cds/appcds/complexURI/ComplexURITest.java line 46:
> 
>> 44: public class ComplexURITest {
>> 45:     final static String listFileName = "test-classlist.txt";
>> 46:     final static String archiveName  = "test-dynamic.jsa";
> 
> You're using the same `archiveName` for both static and dynamic test cases. I'd suggest declare a different archive name for the static case. E.g.
> 
> final static String staticArchiveName = "test-static.jsa";
> final static String dynamicArchiveName = "test-dynamic.jsa";
> 
> Also, I think this test should be excluded from the `hotspot_appcds_dynamic` test group in the `open/test/hotspot/jtreg/TEST.groups` file.

Done. I also added the code to safeguard against `IOException` on Windows in the case of deleting the read-only archive created previously.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20987#discussion_r1772694412


More information about the hotspot-runtime-dev mailing list