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