Looks good assuming the patch was tested with 64bit build.

- Alexey

On 4/30/2020 10:12 AM, Baesken, Matthias wrote:

Hello, please look into this small fix for a link error we faced on Windows 32bit.

Currently we run into this link error :
jpackageapplauncher

* For target support_native_jdk.incubator.jpackage_jpackageapplauncher_BUILD_JPACKAGE_APPLAUNCHEREXE_link:
LINK : fatal error LNK1561: entry point must be defined

looks like the wWinMain signature in
src\jdk.incubator.jpackage\windows\native\applauncher\WinLauncher.cpp
has to be adjusted.

Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8244183

http://cr.openjdk.java.net/~mbaesken/webrevs/8244183.0/

Thanks, Matthias


Reply via email to