+1 .
This fixed it for me when I tried it yesterday.
Advertising
-phil.
On 02/23/2018 06:52 AM, Erik Joelsson wrote:
Looks good and thanks for picking this up! I never got around to it
yesterday.
/Erik
On 2018-02-23 05:24, Magnus Ihse Bursie 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
