Fix parens in ld bootstrap.exp
authorAlan Modra <amodra@gmail.com>
Tue, 26 Jun 2018 04:59:28 +0000 (14:29 +0930)
committerAlan Modra <amodra@gmail.com>
Tue, 26 Jun 2018 11:58:05 +0000 (21:28 +0930)
Seen with tcl 8.5.13:
ERROR: tcl error sourcing .../ld/testsuite/ld-bootstrap/bootstrap.exp.
ERROR: expected boolean value but got " [istarget ia64-*-elf*] || [istarget ia64-*-linux*"
    while executing
"if { "$flags" == "--static" && { [istarget ia64-*-elf*] || [istarget ia64-*-linux*] }
 || [istarget mips*-*-linux*] } {
# On ia64 and mips, tmpdir/l..."

* testsuite/ld-bootstrap/bootstrap.exp: Use parentheses rather
than curly braces in logical expression.

ld/ChangeLog
ld/testsuite/ld-bootstrap/bootstrap.exp

index be9669cbbc2f47e704cae3f51b88c07a7dfbdc43..2065c6ea71fb6583bc8c28665eb64dbcfc8049c7 100644 (file)
@@ -1,3 +1,8 @@
+2018-06-26  Alan Modra  <amodra@gmail.com>
+
+       * testsuite/ld-bootstrap/bootstrap.exp: Use parentheses rather
+       than curly braces in logical expression.
+
 2018-06-26  Nick Clifton  <nickc@redhat.com>
 
        * emultempl/aarch64elf.em: Fix spelling mistake.
index f0bac3f7e9f7f7d413198ad66175a331a7fe1995..5c79e0315ca03fde9fd85c81865b40048e90e071 100644 (file)
@@ -187,8 +187,10 @@ foreach flags $test_flags {
        continue
     }
 
-    if { "$flags" == "--static" && { [istarget ia64-*-elf*] || [istarget ia64-*-linux*] }
-        || [istarget mips*-*-linux*] } {
+    if { "$flags" == "--static"
+        && ([istarget ia64-*-elf*]
+            || [istarget ia64-*-linux*]
+            || [istarget mips*-*-linux*]) } {
        # On ia64 and mips, tmpdir/ld2 != tmpdir/ld3 is normal since they are
        # generated by different linkers, tmpdir/ld1 and tmpdir/ld2.
        # So we rebuild tmpdir/ld2 with tmpdir/ld3.