public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [gccgo] Force "C" locale in gotest
@ 2010-07-20 12:23 Ian Lance Taylor
  2010-07-20 12:28 ` Andreas Schwab
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Lance Taylor @ 2010-07-20 12:23 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 125 bytes --]

This patch forces the use of the "C" locale in gotest, so that egrep
does the right thing.  Committed to gccgo branch.

Ian


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: gotest --]
[-- Type: text/x-diff, Size: 508 bytes --]

diff -r 8f87e072151f libgo/testsuite/gotest
--- a/libgo/testsuite/gotest	Mon Jul 19 01:42:34 2010 -0700
+++ b/libgo/testsuite/gotest	Tue Jul 20 05:21:29 2010 -0700
@@ -8,6 +8,12 @@
 # tests.
 # If files are named on the command line, use them instead of test*.go.
 
+# Makes egrep,grep work better in general if we put them
+# in ordinary C mode instead of what the current language is.
+unset LANG
+export LC_ALL=C
+export LC_CTYPE=C
+
 GC=${GC:-gccgo}
 GL=${GL:-${GC-gccgo}}
 GOLIBS=${GOLIBS:=-lgotesting}

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [gccgo] Force "C" locale in gotest
  2010-07-20 12:23 [gccgo] Force "C" locale in gotest Ian Lance Taylor
@ 2010-07-20 12:28 ` Andreas Schwab
  2010-07-20 13:11   ` Ian Lance Taylor
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Schwab @ 2010-07-20 12:28 UTC (permalink / raw)
  To: Ian Lance Taylor; +Cc: gcc-patches

Ian Lance Taylor <iant@google.com> writes:

> +# Makes egrep,grep work better in general if we put them
> +# in ordinary C mode instead of what the current language is.
> +unset LANG
> +export LC_ALL=C
> +export LC_CTYPE=C

If you already set LC_ALL there is no need to set any other LC_*, since
LC_ALL overrides them all.

Andreas.

-- 
Andreas Schwab, schwab@redhat.com
GPG Key fingerprint = D4E8 DBE3 3813 BB5D FA84  5EC7 45C6 250E 6F00 984E
"And now for something completely different."

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [gccgo] Force "C" locale in gotest
  2010-07-20 12:28 ` Andreas Schwab
@ 2010-07-20 13:11   ` Ian Lance Taylor
  0 siblings, 0 replies; 3+ messages in thread
From: Ian Lance Taylor @ 2010-07-20 13:11 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: gcc-patches

Andreas Schwab <schwab@redhat.com> writes:

> Ian Lance Taylor <iant@google.com> writes:
>
>> +# Makes egrep,grep work better in general if we put them
>> +# in ordinary C mode instead of what the current language is.
>> +unset LANG
>> +export LC_ALL=C
>> +export LC_CTYPE=C
>
> If you already set LC_ALL there is no need to set any other LC_*, since
> LC_ALL overrides them all.

Yeah.  I was just copying code from another version of gotest.  I should
have mentioned that.

Ian

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-07-20 13:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-20 12:23 [gccgo] Force "C" locale in gotest Ian Lance Taylor
2010-07-20 12:28 ` Andreas Schwab
2010-07-20 13:11   ` Ian Lance Taylor

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).