RFR: 8273774: CDSPluginTest should only expect classes_nocoops.jsa exists on supported 64-bit platforms [v5]

Calvin Cheung ccheung at openjdk.java.net
Fri Sep 17 15:12:43 UTC 2021


On Fri, 17 Sep 2021 02:17:09 GMT, Remilia Scarlet <github.com+22913521+1996scarlet at openjdk.org> wrote:

>> The test assumes that it always runs on 64-bit platform and classes_nocoops.jsa is always created.
>> It's a test bug. The test should only expect classes_nocoops.jsa exists if it's running on a supported 64-bit platform.
>> However, for unknown target platform, it's unknown if it's 64-bit or not.
>> 
>> This patch fix the test to check if classes_nocoops.jsa exists only on one of the JDK supported platforms (x64 or aarch64) via the sun.arch.data.model system property.
>> 
>> Please review this change. Thanks!
>
> Remilia Scarlet has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8273774: CDSPluginTest should only expect classes_nocoops.jsa exists on supported 64-bit platforms

Looks good and thanks for fixing this bug.

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

Marked as reviewed by ccheung (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/5519


More information about the core-libs-dev mailing list