--- old/common/autoconf/jdk-version.m4 2015-06-25 15:13:58.693230096 +0200 +++ new/common/autoconf/jdk-version.m4 2015-06-25 15:13:58.601226201 +0200 @@ -85,16 +85,16 @@ 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]+))?(\.([a-zA-Z]+))?((\+)([0-9]+)?(-([-a-zA-Z0-9.]+))?)?$ ]] ]; then + 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 VERSION_MAJOR=${BASH_REMATCH[[1]]} VERSION_MINOR=${BASH_REMATCH[[3]]} VERSION_SECURITY=${BASH_REMATCH[[5]]} VERSION_PATCH=${BASH_REMATCH[[7]]} - VERSION_PRE_BASE=${BASH_REMATCH[[9]]} - VERSION_PRE_DEBUGLEVEL=${BASH_REMATCH[[11]]} - version_plus_separator=${BASH_REMATCH[[13]]} - VERSION_BUILD=${BASH_REMATCH[[14]]} - VERSION_OPT=${BASH_REMATCH[[16]]} + 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]]} # Unspecified numerical fields are interpreted as 0. if test "x$VERSION_MINOR" = x; then VERSION_MINOR=0 @@ -105,7 +105,8 @@ if test "x$VERSION_PATCH" = x; then VERSION_PATCH=0 fi - if test "x$version_plus_separator" != x && test "x$VERSION_BUILD$VERSION_OPT" = x; then + if test "x$version_plus_separator" != x \ + && test "x$VERSION_BUILD$VERSION_OPT_BASE$VERSION_OPT_DEBUGLEVEL" = 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. @@ -116,86 +117,85 @@ fi fi - AC_ARG_WITH(version-pre-base, [AS_HELP_STRING([--with-version-pre-base], + AC_ARG_WITH(version-pre, [AS_HELP_STRING([--with-version-pre], [Set the base part of the version 'PRE' field (pre-release identifier) @<:@'internal'@:>@])], - [with_version_pre_base_present=true], [with_version_pre_base_present=false]) + [with_version_pre_present=true], [with_version_pre_present=false]) - if test "x$with_version_pre_base_present" = xtrue; then - if test "x$with_version_pre_base" = xyes; then - AC_MSG_ERROR([--with-version-pre-base must have a value]) - elif test "x$with_version_pre_base" = xno; then + if test "x$with_version_pre_present" = xtrue; then + if test "x$with_version_pre" = xyes; then + AC_MSG_ERROR([--with-version-pre must have a value]) + elif test "x$with_version_pre" = xno; then # Interpret --without-* as empty string instead of the literal "no" - VERSION_PRE_BASE= + VERSION_PRE= else # Only [a-zA-Z] is allowed in the VERSION_PRE. Outer [ ] to quote m4. - [ VERSION_PRE_BASE=`$ECHO "$with_version_pre_base" | $TR -c -d '[a-z][A-Z]'` ] - if test "x$VERSION_PRE_BASE" != "x$with_version_pre_base"; then - AC_MSG_WARN([--with-version-pre-base value has been sanitized from '$with_version_pre_base' to '$VERSION_PRE_BASE']) + [ VERSION_PRE=`$ECHO "$with_version_pre" | $TR -c -d '[a-z][A-Z]'` ] + if test "x$VERSION_PRE" != "x$with_version_pre"; then + AC_MSG_WARN([--with-version-pre value has been sanitized from '$with_version_pre' to '$VERSION_PRE']) fi fi else if test "x$NO_DEFAULT_VERSION_PARTS" != xtrue; then # Default is to use "internal" as pre - VERSION_PRE_BASE="internal" + VERSION_PRE="internal" fi fi - AC_ARG_WITH(version-pre-debuglevel, [AS_HELP_STRING([--with-version-pre-debuglevel], - [Set the debug level part of the version 'PRE' field (pre-release identifier) @<:@current debug level@:>@])], - [with_version_pre_debuglevel_present=true], [with_version_pre_debuglevel_present=false]) + 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) @<:@..@:>@])], + [with_version_opt_base_present=true], [with_version_opt_base_present=false]) - if test "x$with_version_pre_debuglevel_present" = xtrue; then - if test "x$with_version_pre_debuglevel" = xyes; then - AC_MSG_ERROR([--with-version-pre-debuglevel must have a value]) - elif test "x$with_version_pre_debuglevel" = xno; then + 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 # Interpret --without-* as empty string instead of the literal "no" - VERSION_PRE_DEBUGLEVEL= + VERSION_OPT_BASE= else - # Only [a-zA-Z] is allowed in the VERSION_PRE. Outer [ ] to quote m4. - [ VERSION_PRE_DEBUGLEVEL=`$ECHO "$with_version_pre_debuglevel" | $TR -c -d '[a-z][A-Z]'` ] - if test "x$VERSION_PRE_DEBUGLEVEL" != "x$with_version_pre_debuglevel"; then - AC_MSG_WARN([--with-version-pre-debuglevel value has been sanitized from '$with_version_pre_debuglevel' to '$VERSION_PRE_DEBUGLEVEL']) + # 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']) 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_PRE_DEBUGLEVEL="$DEBUG_LEVEL" - else - VERSION_PRE_DEBUGLEVEL="" - fi + # Default is to calculate a string like this .. + timestamp=`$DATE '+%Y-%m-%d-%H%M%S'` + # 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 - AC_ARG_WITH(version-opt, [AS_HELP_STRING([--with-version-opt], + AC_ARG_WITH(version-opt-debuglevel, [AS_HELP_STRING([--with-version-opt-debuglevel], [Set version 'OPT' field (build metadata) @<:@..@:>@])], - [with_version_opt_present=true], [with_version_opt_present=false]) + [with_version_opt_debuglevel_present=true], [with_version_opt_debuglevel_present=false]) - 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 + 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= + VERSION_OPT_DEBUGLEVEL= 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 - AC_MSG_WARN([--with-version-opt value has been sanitized from '$with_version_opt' to '$VERSION_OPT']) + # 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 calculate a string like this .. - timestamp=`$DATE '+%Y-%m-%d-%H%M%S'` - # 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="$timestamp.$username.$basedirname" + # 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,12 +305,9 @@ # Calculate derived version properties - # Set pre to "pre-base" if debug level is empty (i.e. release), or - # "pre-base.debug-level" otherwise. - if test "x$VERSION_PRE_BASE" = x && test "x$VERSION_PRE_DEBUGLEVEL" != x; then - AC_MSG_ERROR([Cannot set version-pre-debuglevel when version-pre-base is empty]) - fi - VERSION_PRE=$VERSION_PRE_BASE${VERSION_PRE_DEBUGLEVEL:+.$VERSION_PRE_DEBUGLEVEL} + # 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} # VERSION_NUMBER but always with exactly 4 positions, with 0 for empty positions. VERSION_NUMBER_FOUR_POSITIONS=$VERSION_MAJOR.$VERSION_MINOR.$VERSION_SECURITY.$VERSION_PATCH --- old/make/jprt.properties 2015-06-25 15:13:59.077246353 +0200 +++ new/make/jprt.properties 2015-06-25 15:13:58.989242627 +0200 @@ -90,10 +90,10 @@ # Configure args common to all builds # Also allows for additional, testset specific configure arguments to be set jprt.build.configure.args= \ - --with-output-sync=recurse \ - --with-boot-jdk=$ALT_BOOTDIR \ - --with-jobs=$ALT_PARALLEL_COMPILE_JOBS \ - --with-version-opt=$JPRT_JOB_ID \ + --with-output-sync=recurse \ + --with-boot-jdk=$ALT_BOOTDIR \ + --with-jobs=$ALT_PARALLEL_COMPILE_JOBS \ + --with-version-opt-base=$JPRT_JOB_ID \ ${my.additional.build.configure.args.${jprt.test.set}} \ ${my.custom.build.configure.args}