On Tue, 26 Sep 2023 09:53:08 GMT, Jaikiran Pai <j...@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. Christian noted that the `@test` in each of the test files may not even be needed if you could just add the `modules=jdk.jpackage` to the existing content of `TEST.properties` file. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15916#discussion_r1336964564