[PATCH] [8u] Prevent MSDOS8.3 named DLL in built image
Ali Ince
ali.ince at gmail.com
Thu Mar 28 19:47:56 UTC 2019
Hi All,
We're working on building JDK8U with VS2017 at AdoptOpenJDK and found out that the new vcruntime140.dll is copied to the built image named as `vcrunt~1.dll` which is basically because of the extra call to `BASIC_FIXUP_PATH` call in `toolchain_windows.m4` file. If the call is removed, everything works fine.
On previous versions of VS, the VC runtime DLL was originally named in 8.3 style (ex. msvcr100.dll) and BASIC_FIXUP_PATH did not have any affect on the file name itself. I've checked with `toolchain_windows.m4` files in jdk11u and onwards and also saw that this call doesn't exist.
I've came up with the following patch which removes this call. I'm not sure what's the process about updating the checked-in generated_configure.sh but I could also add the patch for that if it's required.
Let me know if you have any feedback or comments.
Regards,
Ali
---------------------------------------------
diff -r e0b7721459ee common/autoconf/toolchain_windows.m4
--- a/common/autoconf/toolchain_windows.m4 Wed Mar 20 16:32:54 2019 +0000
+++ b/common/autoconf/toolchain_windows.m4 Thu Mar 28 00:03:10 2019 +0000
@@ -493,7 +493,6 @@
if $ECHO "$MSVC_DLL_FILETYPE" | $GREP "$CORRECT_MSVCR_ARCH" 2>&1 > /dev/null; then
AC_MSG_RESULT([ok])
MSVC_DLL="$POSSIBLE_MSVC_DLL"
- BASIC_FIXUP_PATH(MSVC_DLL)
AC_MSG_CHECKING([for $DLL_NAME])
AC_MSG_RESULT([$MSVC_DLL])
else
More information about the jdk8u-dev
mailing list