RFR: 8315786: [AIX] Build Disk Local Detection Issue with GNU-utils df on AIX

Erik Joelsson erikj at openjdk.org
Tue Sep 12 16:54:46 UTC 2023


On Wed, 6 Sep 2023 13:46:01 GMT, Deepa Kumari <duke at openjdk.org> wrote:

> Previously [JDK-8304364](https://github.com/openjdk/jdk/pull/13066/files) , the AIX build process raised complaints about the disk location detection, incorrectly determining that the build wasn't on a local disk. However, a partial fix introduced a new problem, the build process consistently reports that it's on a local disk, even when it's not
> The core problem here seems to be that Bash treats quoted commands as string literals, and it only evaluates them if you use 'eval' directly.
>  The  change ensure that Bash correctly evaluates the **DF** variable as a command.
> 
> Reported Issue : [JDK-8315786](https://bugs.openjdk.org/browse/JDK-8315786)

make/autoconf/basic.m4 line 482:

> 480:     # is the same. On older AIXes we just continue to live with a "not local build" warning.
> 481:     if test "x$OPENJDK_TARGET_OS" = xaix; then
> 482:       if ""$DF" -T local > /dev/null 2>&1"; then

Suggestion:

      if $DF -T local > /dev/null 2>&1; then

I can't test this, but I don't think the quotes serve any purpose here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15592#discussion_r1323319043


More information about the build-dev mailing list