I apologize for the extra work my mistake caused you. :( /Magnus
> 23 feb. 2018 kl. 16:19 skrev Volker Simonis <volker.simo...@gmail.com>: > > Good patch, Magnus! > > We've been affected by this as well but instead of digging deeper into > the problem we've installed "readlink" on our AIX build machines, > where it wasn't available until now. Good to know that "readlink" is > still not a mandatory requirement for the build :) > > > On Fri, Feb 23, 2018 at 2:24 PM, Magnus Ihse Bursie > <magnus.ihse.bur...@oracle.com> wrote: >> The if line introduced in JDK-8198318 is missing a "test". >> Thanks to Jon and Erik for discovering the problem and the solution. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8198627 >> Patch inline: >> >> diff --git a/make/autoconf/basics.m4 b/make/autoconf/basics.m4 >> --- a/make/autoconf/basics.m4 >> +++ b/make/autoconf/basics.m4 >> @@ -263,7 +263,7 @@ >> READLINK_TESTED=yes >> fi >> >> - if test "x$READLINK" != x && "x$READLINK_ISGNU" != x; then >> + if test "x$READLINK" != x && test "x$READLINK_ISGNU" != x; then >> $1=`$READLINK -f [$]$1` >> else >> # Save the current directory for restoring afterwards >> >> /Magnus
