From e7ab5e634b5181061083a17741e4e39d4c191bb8 Mon Sep 17 00:00:00 2001 From: Andrew Burgess Date: Fri, 25 Mar 2011 13:08:11 +0000 Subject: Move environment modifications into gdb_init so it's done just once per test, and clear the GREP_OPTIONS environment variable. See, http://sourceware.org/ml/gdb-patches/2011-03/msg00886.html --- gdb/testsuite/ChangeLog | 7 +++++++ gdb/testsuite/lib/gdb.exp | 39 +++++++++++++++++++++------------------ 2 files changed, 28 insertions(+), 18 deletions(-) (limited to 'gdb/testsuite') diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 798f7db..89aaaed 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,10 @@ +2011-03-25 Andrew Burgess + + * lib/gdb.exp (default_gdb_start,gdb_init): Clear the GREP_OPTIONS + environment variable to make grep output more predictable. Move + all the environment setup into gdb_init so it's done once per test + case rather than each time we start gdb. + 2011-03-24 Tom Tromey * gdb.base/label.exp: Use `main:there' linespec. Add re-running diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index 6a2f1a2..a512527 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -1311,25 +1311,9 @@ proc default_gdb_start { } { global gdb_prompt global timeout global gdb_spawn_id; - global env gdb_stop_suppressing_tests; - set env(LC_CTYPE) C - - # Don't let a .inputrc file or an existing setting of INPUTRC mess up - # the test results. Even if /dev/null doesn't exist on the particular - # platform, the readline library will use the default setting just by - # failing to open the file. OTOH, opening /dev/null successfully will - # also result in the default settings being used since nothing will be - # read from this file. - set env(INPUTRC) "/dev/null" - - # The gdb.base/readline.exp arrow key test relies on the standard VT100 - # bindings, so make sure that an appropriate terminal is selected. - # The same bug doesn't show up if we use ^P / ^N instead. - set env(TERM) "vt100" - verbose "Spawning $GDB $INTERNAL_GDBFLAGS $GDBFLAGS" if [info exists gdb_spawn_id] { @@ -2796,11 +2780,30 @@ proc gdb_init { args } { set banned_variables_traced 1 } - # We set LC_ALL and LANG to C so that we get the same messages as - # expected. + # We set LC_ALL, LC_CTYPE, and LANG to C so that we get the same + # messages as expected. setenv LC_ALL C + setenv LC_CTYPE C setenv LANG C + # Don't let a .inputrc file or an existing setting of INPUTRC mess up + # the test results. Even if /dev/null doesn't exist on the particular + # platform, the readline library will use the default setting just by + # failing to open the file. OTOH, opening /dev/null successfully will + # also result in the default settings being used since nothing will be + # read from this file. + setenv INPUTRC "/dev/null" + + # The gdb.base/readline.exp arrow key test relies on the standard VT100 + # bindings, so make sure that an appropriate terminal is selected. + # The same bug doesn't show up if we use ^P / ^N instead. + setenv TERM "vt100" + + # Some tests (for example gdb.base/maint.exp) shell out from gdb to use + # grep. Clear GREP_OPTIONS to make the behavoiur predictable, + # especially having color output turned on can cause tests to fail. + setenv GREP_OPTIONS "" + return [eval default_gdb_init $args]; } -- cgit v1.1