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

Mandy Chung mchung at openjdk.java.net
Fri Sep 17 02:23:46 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.

Thanks for the change.

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

Marked as reviewed by mchung (Reviewer).

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


More information about the core-libs-dev mailing list