RFR: 8345259: Disallow ALL-MODULE-PATH without explicit --module-path [v5]

Severin Gehwolf sgehwolf at openjdk.org
Wed Dec 11 15:21:14 UTC 2024


On Tue, 10 Dec 2024 23:50:28 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> Severin Gehwolf has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Handle non-existent module-path with ALL-MODULE-PATH
>>  - Move test, more test fixes for JEP 493 enabled builds
>
> test/jdk/tools/jlink/basic/AllModulePathTest.java line 54:
> 
>> 52:  * @run main/othervm -Xmx1g AllModulePathTest
>> 53:  */
>> 54: public class AllModulePathTest {
> 
> Any reason why these test cases can't be added to `AllModulePath`?  `AllModulePathTest` and `AllModulePath` test names are very alike.

I'll merge the tests into `AllModulePath`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22494#discussion_r1880397567


More information about the core-libs-dev mailing list