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

Magnus Ihse Bursie ihse at openjdk.java.net
Mon Mar 7 19:56:58 UTC 2022


On Mon, 7 Mar 2022 19:53:34 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> make/autoconf/jdk-version.m4 line 129:
>> 
>>> 127:     AC_MSG_ERROR([--with-vendor-url must have a value])
>>> 128:   elif test "x$with_vendor_url" = xno; then
>>> 129:     AC_MSG_WARN([--without-vendor-url is the same as not passing --with-vendor-url to begin with])
>> 
>> Same as with vendor.
>
> I think this, even more, makes it clear that `--without-vendor-url` can´t possible be meant to be interpreted as "use https://openjdk.java.net/". 
> 
> Basically, I think what I'm arguing for is that we can fold this check into the "yes" check -- both `--with-vendor-url` (with no given value) and `--without-vendor-url` are invalid. So something like this:
> 
> 
>  if test "x$with_vendor_url" = xyes || test "x$with_vendor_url" = xno; then
>     AC_MSG_ERROR([--with-vendor-url must have a value])
>   elif...

... and this goes for all the changes in the PR.

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

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



More information about the build-dev mailing list