On Thu, 10 Mar 2022 09:41:32 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
>> Julian Waters has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix syntax errors > > make/autoconf/jdk-version.m4 line 554: > >> 552: elif test "x$with_macosx_bundle_build_version" = xno; then >> 553: # Interpret --without-* as empty string instead of the literal "no" >> 554: MACOSX_BUNDLE_BUILD_VERSION= > > Same objection here. And even if it works right now, if we do allow this, it > means that all future changes in the build system that involves > MACOSX_BUNDLE_BUILD_VERSION needs to be verified that it works with the empty > string. >From what I can tell, wouldn't MACOSX_BUNDLE_BUILD_VERSION be an empty string >anyway if both VERSION_BUILD and VERSION_OPT = ""? ------------- PR: https://git.openjdk.java.net/jdk/pull/7713