changeset 2129:21efe3b23a5a

8139951: Do not store debug level in OPT part of Verona string Reviewed-by: erikj
author ihse
date Tue, 20 Oct 2015 12:12:43 +0200
parents 83a19847a443
children 3f5bfe602a9f
files common/autoconf/generated-configure.sh common/autoconf/jdk-version.m4
diffstat 2 files changed, 45 insertions(+), 120 deletions(-) [+]
line wrap: on
line diff
--- a/common/autoconf/generated-configure.sh	Mon Oct 12 10:31:23 2015 -0400
+++ b/common/autoconf/generated-configure.sh	Tue Oct 20 12:12:43 2015 +0200
@@ -1070,8 +1070,7 @@
 with_build_number
 with_version_string
 with_version_pre
-with_version_opt_base
-with_version_opt_debuglevel
+with_version_opt
 with_version_build
 with_version_major
 with_version_minor
@@ -1929,11 +1928,7 @@
   --with-version-string   Set version string [calculated]
   --with-version-pre      Set the base part of the version 'PRE' field
                           (pre-release identifier) ['internal']
-  --with-version-opt-base Set version 'OPT' base field. Debug level will be
-                          appended. (build metadata)
-                          [<timestamp>.<user>.<dirname>]
-  --with-version-opt-debuglevel
-                          Set version 'OPT' field (build metadata)
+  --with-version-opt      Set version 'OPT' field (build metadata)
                           [<timestamp>.<user>.<dirname>]
   --with-version-build    Set version 'BUILD' field (build number) [not
                           specified]
@@ -4436,7 +4431,7 @@
 #CUSTOM_AUTOCONF_INCLUDE
 
 # Do not change or remove the following line, it is needed for consistency checks:
-DATE_WHEN_GENERATED=1444104121
+DATE_WHEN_GENERATED=1445335893
 
 ###############################################################################
 #
@@ -20449,7 +20444,7 @@
     as_fn_error $? "--with-version-string must have a value" "$LINENO" 5
   elif test "x$with_version_string" != x; then
     # Additional [] needed to keep m4 from mangling shell constructs.
-    if  [[ $with_version_string =~ ^([0-9]+)(\.([0-9]+))?(\.([0-9]+))?(\.([0-9]+))?(-([a-zA-Z]+))?((\+)([0-9]+)?(-([-a-zA-Z0-9.]+))?(_([a-zA-Z]+))?)?$ ]] ; then
+    if  [[ $with_version_string =~ ^([0-9]+)(\.([0-9]+))?(\.([0-9]+))?(\.([0-9]+))?(-([a-zA-Z]+))?((\+)([0-9]+)?(-([-a-zA-Z0-9.]+))?)?$ ]] ; then
       VERSION_MAJOR=${BASH_REMATCH[1]}
       VERSION_MINOR=${BASH_REMATCH[3]}
       VERSION_SECURITY=${BASH_REMATCH[5]}
@@ -20457,8 +20452,7 @@
       VERSION_PRE=${BASH_REMATCH[9]}
       version_plus_separator=${BASH_REMATCH[11]}
       VERSION_BUILD=${BASH_REMATCH[12]}
-      VERSION_OPT_BASE=${BASH_REMATCH[14]}
-      VERSION_OPT_DEBUGLEVEL=${BASH_REMATCH[16]}
+      VERSION_OPT=${BASH_REMATCH[14]}
       # Unspecified numerical fields are interpreted as 0.
       if test "x$VERSION_MINOR" = x; then
         VERSION_MINOR=0
@@ -20470,7 +20464,7 @@
         VERSION_PATCH=0
       fi
       if test "x$version_plus_separator" != x \
-          && test "x$VERSION_BUILD$VERSION_OPT_BASE$VERSION_OPT_DEBUGLEVEL" = x; then
+          && test "x$VERSION_BUILD$VERSION_OPT" = x; then
         as_fn_error $? "Version string contains + but both 'BUILD' and 'OPT' are missing" "$LINENO" 5
       fi
       # Stop the version part process from setting default values.
@@ -20512,26 +20506,26 @@
   fi
 
 
-# Check whether --with-version-opt-base was given.
-if test "${with_version_opt_base+set}" = set; then :
-  withval=$with_version_opt_base; with_version_opt_base_present=true
-else
-  with_version_opt_base_present=false
-fi
-
-
-  if test "x$with_version_opt_base_present" = xtrue; then
-    if test "x$with_version_opt_base" = xyes; then
-      as_fn_error $? "--with-version-opt-base must have a value" "$LINENO" 5
-    elif test "x$with_version_opt_base" = xno; then
+# Check whether --with-version-opt was given.
+if test "${with_version_opt+set}" = set; then :
+  withval=$with_version_opt; with_version_opt_present=true
+else
+  with_version_opt_present=false
+fi
+
+
+  if test "x$with_version_opt_present" = xtrue; then
+    if test "x$with_version_opt" = xyes; then
+      as_fn_error $? "--with-version-opt must have a value" "$LINENO" 5
+    elif test "x$with_version_opt" = xno; then
       # Interpret --without-* as empty string instead of the literal "no"
-      VERSION_OPT_BASE=
-    else
-      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT_BASE. Outer [ ] to quote m4.
-       VERSION_OPT_BASE=`$ECHO "$with_version_opt_base" | $TR -c -d '[a-z][A-Z][0-9].-'`
-      if test "x$VERSION_OPT_BASE" != "x$with_version_opt_base"; then
-        { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: --with-version-opt-base value has been sanitized from '$with_version_opt_base' to '$VERSION_OPT_BASE'" >&5
-$as_echo "$as_me: WARNING: --with-version-opt-base value has been sanitized from '$with_version_opt_base' to '$VERSION_OPT_BASE'" >&2;}
+      VERSION_OPT=
+    else
+      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT. Outer [ ] to quote m4.
+       VERSION_OPT=`$ECHO "$with_version_opt" | $TR -c -d '[a-z][A-Z][0-9].-'`
+      if test "x$VERSION_OPT" != "x$with_version_opt"; then
+        { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: --with-version-opt value has been sanitized from '$with_version_opt' to '$VERSION_OPT'" >&5
+$as_echo "$as_me: WARNING: --with-version-opt value has been sanitized from '$with_version_opt' to '$VERSION_OPT'" >&2;}
       fi
     fi
   else
@@ -20541,43 +20535,10 @@
       # Outer [ ] to quote m4.
        username=`$ECHO "$USER" | $TR -d -c '[a-z][A-Z][0-9]'`
        basedirname=`$BASENAME "$TOPDIR" | $TR -d -c '[a-z][A-Z][0-9].-'`
-      VERSION_OPT_BASE="$timestamp.$username.$basedirname"
-    fi
-  fi
-
-
-# Check whether --with-version-opt-debuglevel was given.
-if test "${with_version_opt_debuglevel+set}" = set; then :
-  withval=$with_version_opt_debuglevel; with_version_opt_debuglevel_present=true
-else
-  with_version_opt_debuglevel_present=false
-fi
-
-
-  if test "x$with_version_opt_debuglevel_present" = xtrue; then
-    if test "x$with_version_opt_debuglevel" = xyes; then
-      as_fn_error $? "--with-version-opt-debuglevel must have a value" "$LINENO" 5
-    elif test "x$with_version_opt_debuglevel" = xno; then
-      # Interpret --without-* as empty string instead of the literal "no"
-      VERSION_OPT_DEBUGLEVEL=
-    else
-      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT_DEBUGLEVEL. Outer [ ] to quote m4.
-       VERSION_OPT_DEBUGLEVEL=`$ECHO "$with_version_opt_debuglevel" | $TR -c -d '[a-z][A-Z][0-9].-'`
-      if test "x$VERSION_OPT_DEBUGLEVEL" != "x$with_version_opt_debuglevel"; then
-        { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: --with-version-opt-debuglevel value has been sanitized from '$with_version_opt_debuglevel' to '$VERSION_OPT_DEBUGLEVEL'" >&5
-$as_echo "$as_me: WARNING: --with-version-opt-debuglevel value has been sanitized from '$with_version_opt_debuglevel' to '$VERSION_OPT_DEBUGLEVEL'" >&2;}
-      fi
-    fi
-  else
-    if test "x$NO_DEFAULT_VERSION_PARTS" != xtrue; then
-      # Default is to use the debug level name, except for release which is empty.
-      if test "x$DEBUG_LEVEL" != "xrelease"; then
-        VERSION_OPT_DEBUGLEVEL="$DEBUG_LEVEL"
-      else
-        VERSION_OPT_DEBUGLEVEL=""
-      fi
-    fi
-  fi
+      VERSION_OPT="$timestamp.$username.$basedirname"
+    fi
+  fi
+
 
 # Check whether --with-version-build was given.
 if test "${with_version_build+set}" = set; then :
@@ -20812,10 +20773,6 @@
 
   # Calculate derived version properties
 
-  # Set opt to "opt-base" if debug level is empty (i.e. release), or
-  # "opt-base_debug-level" otherwise.
-  VERSION_OPT=$VERSION_OPT_BASE${VERSION_OPT_DEBUGLEVEL:+_$VERSION_OPT_DEBUGLEVEL}
-
   # Set VERSION_IS_GA based on if VERSION_PRE has a value
   if test "x$VERSION_PRE" = x; then
     VERSION_IS_GA=true
--- a/common/autoconf/jdk-version.m4	Mon Oct 12 10:31:23 2015 -0400
+++ b/common/autoconf/jdk-version.m4	Tue Oct 20 12:12:43 2015 +0200
@@ -85,7 +85,7 @@
     AC_MSG_ERROR([--with-version-string must have a value])
   elif test "x$with_version_string" != x; then
     # Additional [] needed to keep m4 from mangling shell constructs.
-    if [ [[ $with_version_string =~ ^([0-9]+)(\.([0-9]+))?(\.([0-9]+))?(\.([0-9]+))?(-([a-zA-Z]+))?((\+)([0-9]+)?(-([-a-zA-Z0-9.]+))?(_([a-zA-Z]+))?)?$ ]] ]; then
+    if [ [[ $with_version_string =~ ^([0-9]+)(\.([0-9]+))?(\.([0-9]+))?(\.([0-9]+))?(-([a-zA-Z]+))?((\+)([0-9]+)?(-([-a-zA-Z0-9.]+))?)?$ ]] ]; then
       VERSION_MAJOR=${BASH_REMATCH[[1]]}
       VERSION_MINOR=${BASH_REMATCH[[3]]}
       VERSION_SECURITY=${BASH_REMATCH[[5]]}
@@ -93,8 +93,7 @@
       VERSION_PRE=${BASH_REMATCH[[9]]}
       version_plus_separator=${BASH_REMATCH[[11]]}
       VERSION_BUILD=${BASH_REMATCH[[12]]}
-      VERSION_OPT_BASE=${BASH_REMATCH[[14]]}
-      VERSION_OPT_DEBUGLEVEL=${BASH_REMATCH[[16]]}
+      VERSION_OPT=${BASH_REMATCH[[14]]}
       # Unspecified numerical fields are interpreted as 0.
       if test "x$VERSION_MINOR" = x; then
         VERSION_MINOR=0
@@ -106,7 +105,7 @@
         VERSION_PATCH=0
       fi
       if test "x$version_plus_separator" != x \
-          && test "x$VERSION_BUILD$VERSION_OPT_BASE$VERSION_OPT_DEBUGLEVEL" = x; then
+          && test "x$VERSION_BUILD$VERSION_OPT" = x; then
         AC_MSG_ERROR([Version string contains + but both 'BUILD' and 'OPT' are missing])
       fi
       # Stop the version part process from setting default values.
@@ -141,21 +140,21 @@
     fi
   fi
 
-  AC_ARG_WITH(version-opt-base, [AS_HELP_STRING([--with-version-opt-base],
-      [Set version 'OPT' base field. Debug level will be appended. (build metadata) @<:@<timestamp>.<user>.<dirname>@:>@])],
-      [with_version_opt_base_present=true], [with_version_opt_base_present=false])
+  AC_ARG_WITH(version-opt, [AS_HELP_STRING([--with-version-opt],
+      [Set version 'OPT' field (build metadata) @<:@<timestamp>.<user>.<dirname>@:>@])],
+      [with_version_opt_present=true], [with_version_opt_present=false])
 
-  if test "x$with_version_opt_base_present" = xtrue; then
-    if test "x$with_version_opt_base" = xyes; then
-      AC_MSG_ERROR([--with-version-opt-base must have a value])
-    elif test "x$with_version_opt_base" = xno; then
+  if test "x$with_version_opt_present" = xtrue; then
+    if test "x$with_version_opt" = xyes; then
+      AC_MSG_ERROR([--with-version-opt must have a value])
+    elif test "x$with_version_opt" = xno; then
       # Interpret --without-* as empty string instead of the literal "no"
-      VERSION_OPT_BASE=
+      VERSION_OPT=
     else
-      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT_BASE. Outer [ ] to quote m4.
-      [ VERSION_OPT_BASE=`$ECHO "$with_version_opt_base" | $TR -c -d '[a-z][A-Z][0-9].-'` ]
-      if test "x$VERSION_OPT_BASE" != "x$with_version_opt_base"; then
-        AC_MSG_WARN([--with-version-opt-base value has been sanitized from '$with_version_opt_base' to '$VERSION_OPT_BASE'])
+      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT. Outer [ ] to quote m4.
+      [ VERSION_OPT=`$ECHO "$with_version_opt" | $TR -c -d '[a-z][A-Z][0-9].-'` ]
+      if test "x$VERSION_OPT" != "x$with_version_opt"; then
+        AC_MSG_WARN([--with-version-opt value has been sanitized from '$with_version_opt' to '$VERSION_OPT'])
       fi
     fi
   else
@@ -165,37 +164,10 @@
       # Outer [ ] to quote m4.
       [ username=`$ECHO "$USER" | $TR -d -c '[a-z][A-Z][0-9]'` ]
       [ basedirname=`$BASENAME "$TOPDIR" | $TR -d -c '[a-z][A-Z][0-9].-'` ]
-      VERSION_OPT_BASE="$timestamp.$username.$basedirname"
+      VERSION_OPT="$timestamp.$username.$basedirname"
     fi
   fi
 
-  AC_ARG_WITH(version-opt-debuglevel, [AS_HELP_STRING([--with-version-opt-debuglevel],
-      [Set version 'OPT' field (build metadata) @<:@<timestamp>.<user>.<dirname>@:>@])],
-      [with_version_opt_debuglevel_present=true], [with_version_opt_debuglevel_present=false])
-
-  if test "x$with_version_opt_debuglevel_present" = xtrue; then
-    if test "x$with_version_opt_debuglevel" = xyes; then
-      AC_MSG_ERROR([--with-version-opt-debuglevel must have a value])
-    elif test "x$with_version_opt_debuglevel" = xno; then
-      # Interpret --without-* as empty string instead of the literal "no"
-      VERSION_OPT_DEBUGLEVEL=
-    else
-      # Only [-.a-zA-Z0-9] is allowed in the VERSION_OPT_DEBUGLEVEL. Outer [ ] to quote m4.
-      [ VERSION_OPT_DEBUGLEVEL=`$ECHO "$with_version_opt_debuglevel" | $TR -c -d '[a-z][A-Z][0-9].-'` ]
-      if test "x$VERSION_OPT_DEBUGLEVEL" != "x$with_version_opt_debuglevel"; then
-        AC_MSG_WARN([--with-version-opt-debuglevel value has been sanitized from '$with_version_opt_debuglevel' to '$VERSION_OPT_DEBUGLEVEL'])
-      fi
-    fi
-  else
-    if test "x$NO_DEFAULT_VERSION_PARTS" != xtrue; then
-      # Default is to use the debug level name, except for release which is empty.
-      if test "x$DEBUG_LEVEL" != "xrelease"; then
-        VERSION_OPT_DEBUGLEVEL="$DEBUG_LEVEL"
-      else
-        VERSION_OPT_DEBUGLEVEL=""
-      fi
-    fi
-  fi
   AC_ARG_WITH(version-build, [AS_HELP_STRING([--with-version-build],
       [Set version 'BUILD' field (build number) @<:@not specified@:>@])],
       [with_version_build_present=true], [with_version_build_present=false])
@@ -305,10 +277,6 @@
 
   # Calculate derived version properties
 
-  # Set opt to "opt-base" if debug level is empty (i.e. release), or
-  # "opt-base_debug-level" otherwise.
-  VERSION_OPT=$VERSION_OPT_BASE${VERSION_OPT_DEBUGLEVEL:+_$VERSION_OPT_DEBUGLEVEL}
-
   # Set VERSION_IS_GA based on if VERSION_PRE has a value
   if test "x$VERSION_PRE" = x; then
     VERSION_IS_GA=true