RFR: JDK-8282700: Properly handle several --without options during configure [v4]

[Julian Waters]

On Thu, 10 Mar 2022 09:40:13 GMT, Magnus Ihse Bursie <ihse at 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 507:
> 
>> 505:     # Set vendor version string if --without is not passed
>> 506:     # Check not required if an empty value is passed, since VENDOR_VERSION_STRING
>> 507:     # would then be set to ""
> 
> I'm not sure that works. In general, values like these are *supposed* to have a value. The default fallback is "N/A", which is not the empty string. I see no point in allowing overriding this to be empty. It just introduces cases where the build might fail later, for something that will hardly ever be tested.

I treated this differently since it didn't seem to have any default value at all, as opposed to the other vendor options. I may have made a mistake while looking for where the vendor version string is defined in this case

-------------

PR: https://git.openjdk.java.net/jdk/pull/7713