RFR: 8253426: jpackage is unable to generate working EXE for add-launcher … [v3]
Andy Herrick
herrick at openjdk.java.net
Fri Sep 25 21:28:48 UTC 2020
On Thu, 24 Sep 2020 21:43:52 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> Andy Herrick has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8253426: jpackage is unable to generate working EXE for add-launcher configurations
>
> test/jdk/tools/jpackage/share/AddLauncherTest.java line 218:
>
>> 216:
>> 217: TKit.trace("moduleValue: " + moduleValue + " mainClass: " + mainClass
>> 218: + " classpath: " + classpath);
>
> `classpath` variable is `null` at this point. What is the point to log its value? The whole log statement is excessive
> because `classpath` variable is `null` and value of `moduleValue` variable is logged implicitly in TKit.assertEquals()
> call.
ok - implemented as you suggested
-------------
PR: https://git.openjdk.java.net/jdk/pull/347
More information about the core-libs-dev
mailing list