Integrated: 8308585: AC_REQUIRE: `PLATFORM_EXTRACT_TARGET_AND_BUILD' was expanded before it was required

[Erik Joelsson]

On Tue, 13 Jun 2023 21:47:28 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> Autoconf is emitting this warning after [JDK-8307573](https://bugs.openjdk.org/browse/JDK-8307573):
> 
> stdin:85: warning: AC_REQUIRE: `PLATFORM_EXTRACT_TARGET_AND_BUILD' was expanded before it was required
> stdin:85: http://www.gnu.org/software/autoconf/manual/autoconf.html#Expanded-Before-Required
> /home/erik/git/jdk/open/make/autoconf/platform.m4:644: PLATFORM_SETUP_OPENJDK_BUILD_AND_TARGET is expanded from...
> stdin:85: the top level
> 
> This is caused by using `AC_DEFUN_ONCE` and `AC_REQUIRE` in the wrong way. In the OpenJDK configure script, we use a more imperative model of explicitly calling `AC_DEFUN` macros in most situations. This is mostly done due to how easy it is to get the other model wrong, especially when the models are mixed. The simple solution here is to just change `PLATFORM_CHECK_DEPRECATION` to an `AC_DEFUN` macro and remove the `AC_REQUIRE` call.
> 
> When verifying the fix, I noticed that the check isn't currently working. At least I am still able to configure windows-x86 with no warning or error. With this fix, the error is printed as expected. This makes me wonder if the original change ever worked.
> 
> I'm also adding the deprecation override configure arg to jib-profiles.js to make it possible to still configure windows-x86 using Jib at Oracle. We aren't doing that regularly, but as long as the configuration is there, it should at least work.

This pull request has now been integrated.

Changeset: 554b4d72
Author:    Erik Joelsson <erikj at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/554b4d7243f2c7bb3cc3260358fc7c3c636d7536
Stats:     5 lines in 2 files changed: 1 ins; 1 del; 3 mod

8308585: AC_REQUIRE: `PLATFORM_EXTRACT_TARGET_AND_BUILD' was expanded before it was required

Reviewed-by: mikael

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

PR: https://git.openjdk.org/jdk/pull/14459