On Wed, 15 Sep 2021 06:28:25 GMT, Remilia Scarlet 
<github.com+22913521+1996scar...@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 <sunx...@loongson.cn>
Committer: Calvin Cheung <cche...@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

Reply via email to