On Tue, 26 Sep 2023 09:40:11 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

> AIX currently does not have the jdk.jpackage system module. We have to take 
> this into account for these jpackage tests.

test/jdk/tools/jpackage/junit/jdk.jpackage/jdk/jpackage/internal/AppImageFileTest.java
 line 26:

> 24: /*
> 25:  * @test
> 26:  * @modules jdk.jpackage

Hello Matthias,

Would moving `@modules jdk.jpackage` to 
`test/jdk/tools/jpackage/junit/TEST.properties` like the following:


modules=jdk.jpackage


and adding the `@test` declaration to these  tests, get this passing on AIX? 
That way it will avoid the repeated `@modules` declaration in these test files.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15916#discussion_r1336954393

Reply via email to