RFR (S): 8217305: Missing 0 in java.dll file version cause issues with patch management software

Kevin Walls kevin.walls at oracle.com
Tue Mar 12 07:57:11 UTC 2019


Hi,

I'd like to get a review for an 8u fix for:

8217305: Missing 0 in java.dll file version cause issues with patch 
management software
JBS:https://bugs.openjdk.java.net/browse/JDK-8217305

Diff pasted below as it is a one-liner (plus regenerating 
common/autoconf/generated-configure.sh).

We need to correct the JDK_FVER setting in flags.m4, which dropped some 
use of "COOKED" version during a previous change. Important to change 
back as this is a regression, breaking the version format expected by 
some people and tools.

Low risk, manual testing.

Thanks
Kevin


bash-4.2$ hg diff
diff -r 9da665f87c4b common/autoconf/flags.m4
--- a/common/autoconf/flags.m4  Wed Oct 25 13:11:07 2017 -0700
+++ b/common/autoconf/flags.m4  Mon Mar 11 15:48:15 2019 -0700
@@ -111,7 +111,7 @@
          -d 
\"JDK_VER=\$(JDK_MINOR_VERSION).\$(JDK_MICRO_VERSION).\$(COOKED_JDK_UPDATE_VERSION).\$(COOKED_BUILD_NUMBER)\" 
\
          -d \"JDK_COPYRIGHT=Copyright \xA9 $COPYRIGHT_YEAR\" \
          -d \"JDK_NAME=\$(PRODUCT_NAME) \$(JDK_RC_PLATFORM_NAME) 
\$(JDK_MINOR_VERSION) \$(JDK_UPDATE_META_TAG)\" \
-        -d \"JDK_FVER=\$(JDK_MINOR_VERSION),\$(JDK_MICRO_VERSION),\$(if 
\$(JDK_UPDATE_VERSION),\$(JDK_UPDATE_VERSION),0),\$(COOKED_BUILD_NUMBER)\""
+        -d \"JDK_FVER=\$(JDK_MINOR_VERSION),\$(JDK_MICRO_VERSION),\$(if 
\$(COOKED_JDK_UPDATE_VERSION),\$(COOKED_JDK_UPDATE_VERSION),0),\$(COOKED_BUILD_NUMBER)\""
    fi
    AC_SUBST(RC_FLAGS)

diff -r 9da665f87c4b common/autoconf/generated-configure.sh
--- a/common/autoconf/generated-configure.sh    Wed Oct 25 13:11:07 2017 
-0700
+++ b/common/autoconf/generated-configure.sh    Mon Mar 11 15:48:15 2019 
-0700
@@ -4358,7 +4358,7 @@
  #CUSTOM_AUTOCONF_INCLUDE

  # Do not change or remove the following line, it is needed for 
consistency checks:
-DATE_WHEN_GENERATED=1551441508
+DATE_WHEN_GENERATED=1552344461

  ###############################################################################
  #
@@ -40540,7 +40540,7 @@
          -d 
\"JDK_VER=\$(JDK_MINOR_VERSION).\$(JDK_MICRO_VERSION).\$(COOKED_JDK_UPDATE_VERSION).\$(COOKED_BUILD_NUMBER)\" 
\
          -d \"JDK_COPYRIGHT=Copyright \xA9 $COPYRIGHT_YEAR\" \
          -d \"JDK_NAME=\$(PRODUCT_NAME) \$(JDK_RC_PLATFORM_NAME) 
\$(JDK_MINOR_VERSION) \$(JDK_UPDATE_META_TAG)\" \
-        -d \"JDK_FVER=\$(JDK_MINOR_VERSION),\$(JDK_MICRO_VERSION),\$(if 
\$(JDK_UPDATE_VERSION),\$(JDK_UPDATE_VERSION),0),\$(COOKED_BUILD_NUMBER)\""
+        -d \"JDK_FVER=\$(JDK_MINOR_VERSION),\$(JDK_MICRO_VERSION),\$(if 
\$(COOKED_JDK_UPDATE_VERSION),\$(COOKED_JDK_UPDATE_VERSION),0),\$(COOKED_BUILD_NUMBER)\""
    fi





More information about the jdk8u-dev mailing list