[gdb/testsuite] Fix ls_host return in index-cache.exp
authorTom de Vries <tdevries@suse.de>
Tue, 7 May 2019 09:52:17 +0000 (11:52 +0200)
committerTom de Vries <tdevries@suse.de>
Tue, 7 May 2019 09:52:17 +0000 (11:52 +0200)
When adding a debug print here in index-cache.exp:
...
 proc_with_prefix test_cache_disabled { cache_dir } {
     lassign [ls_host $cache_dir] ret files_before
+    puts "before: '$files_before'"
+    exit
...
we have:
...
files_before: ''
...

When further adding:
...
 proc_with_prefix test_cache_disabled { cache_dir } {
+    exec touch $cache_dir/foo.1 $cache_dir/foo.2 $cache_dir/foo.3
...
we have:
...
files_before: 'foo.1'
...
while we're expecting file_before to contain foo.[123].

Fix this by making the return statement in ls_host return a list rather than a
string (in accordance with the ls_host documentation), after which we have:
...
files_before: 'foo.1 foo.2 foo.3'
...

Tested on x86_64-linux.

gdb/testsuite/ChangeLog:

2019-05-07  Tom de Vries  <tdevries@suse.de>

* gdb.base/index-cache.exp (ls_host): Fix return statement.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/index-cache.exp

index 0af0d37..9610579 100644 (file)
@@ -1,5 +1,9 @@
 2019-05-07  Tom de Vries  <tdevries@suse.de>
 
+       * gdb.base/index-cache.exp (ls_host): Fix return statement.
+
+2019-05-07  Tom de Vries  <tdevries@suse.de>
+
        PR testsuite/24522
        * gdb.dlang/watch-loc.c: Fix debug_info_offset in .debug_aranges
        section.
index 5baba84..b76a02b 100644 (file)
@@ -50,7 +50,7 @@ proc ls_host { dir } {
        }
     }
 
-    return "0 $filtered"
+    return [list 0 $filtered]
 }
 
 # Execute "show index-cache stats" and verify the output against expected