Integrated: 8273774: CDSPluginTest should only expect classes_nocoops.jsa exists on supported 64-bit platforms

Remilia Scarlet github.com+22913521+1996scarlet at openjdk.java.net
Fri Sep 17 16:45:47 UTC 2021


On Wed, 15 Sep 2021 06:28:25 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!

This pull request has now been integrated.

Changeset: 83020617
Author:    sunxu <sunxu01 at loongson.cn>
Committer: Calvin Cheung <ccheung at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/83020617e70d40a42029720534af561f8af8bce2
Stats:     13 lines in 1 file changed: 11 ins; 0 del; 2 mod

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

Reviewed-by: ccheung, mchung

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

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


More information about the core-libs-dev mailing list