RFR: 8359181: Error messages generated by configure --help after 8301197

Erik Joelsson erikj at openjdk.org
Wed Jun 11 08:07:28 UTC 2025


On Wed, 11 Jun 2025 03:21:06 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
> 
> I have no idea why in make function `HELP_PRINT_ADDITIONAL_HELP_AND_EXIT` can use `$PRINTF` variable but can not use `$ECHO` varibale. This PR use `$PRINTF` instead of `$ECHO`.
> 
> Change has been verified locally, almost no risk.

This does not seem like the correct solution. This macro is only called directly from `make/autoconf/configure`, which is our wrapper script around the autoconf generated scripts. That wrapper expects this macro to be using `printf` so it sets `PRINTF=printf` on the command line:
https://github.com/openjdk/jdk/blob/9586817cea3f1cad8a49d43e9106e25dafa04765/make/autoconf/configure#L369
In JDK-8301197, we changed most uses of `printf` to `echo`, but this explicit call wasn't updated to also set `ECHO=echo`. Doing that fixes the issue.

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

PR Review: https://git.openjdk.org/jdk/pull/25740#pullrequestreview-2916080514


More information about the build-dev mailing list