From: Andrew Burgess Date: Fri, 25 Mar 2011 13:08:11 +0000 (+0000) Subject: Move environment modifications into gdb_init so it's done just once per X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e7ab5e634b5181061083a17741e4e39d4c191bb8;p=binutils-gdb.git 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 --- diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 798f7db532e..89aaaed8c1c 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 6a2f1a22061..a512527d79f 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]; }