[FFmpeg-devel] [PATCH 1/3] Set LC_ALL to C at the start of the configure run.

[Måns Rullgård]

Diego 'Flameeyes' Petten? wrote:
> Since different locales can output different messages, which makes
> troubleshooting harder, and makes it impossible to grep for specific
> messages (like compiler warnings) during testing, reset the locale to
> the default (C) locale at the beginning of the script.
> ---
>
>  configure |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/configure b/configure
> index 0fb0721..34512f8 100755
> --- a/configure
> +++ b/configure
> @@ -7,6 +7,10 @@
>  # Copyright (c) 2005-2006 Mans Rullgard
>  #
>
> +# Use standard locale, to make sure the messages are in English,
> +# rather than system locale
> +export LC_ALL=C

Closer, but still only a cigarette.  This is not valid POSIX shell
syntax.  The export must be separate from the assignment.  The
order is irrelevant, but I prefer having the assignment first.

-- 
M?ns Rullg?rd
mans at mansr.com