Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Ingo Molnar
* Priit Laes <[EMAIL PROTECTED]> wrote: > sed-vdsosym := -e 's/^00*/0/' \ > - -e 's/^\([0-9a-fA-F]*\) . \(VDSO[a-zA-Z0-9_]*\)$$/\2 = 0x\1;/p' > + -e 's/^\([[:xdigit:]]*\) . \(VDSO[[:alnum:]_]*\)$$/\2 = 0x\1;/p' thanks, i picked up this one instead of the first version.

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Priit Laes
Ühel kenal päeval, P, 2008-02-24 kell 10:03, kirjutas Sam Ravnborg: > Hi Prit. > > Could we just use [[:alnum:]] and drop the setting of LC_ALL? Yes, it works. Reverted the previous patch and did this: diff --git a/arch/x86/vdso/Makefile b/arch/x86/vdso/Makefile index f385a4b..b8bd0c4 100644

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Sam Ravnborg
Hi Prit. Could we just use [[:alnum:]] and drop the setting of LC_ALL? Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Ingo Molnar
* Priit Laes <[EMAIL PROTECTED]> wrote: > x86: fix build on some non-C locales[1]. > > For some locales regex range [a-zA-Z] does not work as it is supposed > to so we have to specify LANG=C to make it work as intended. thanks, applied. Ingo -- To unsubscribe from this list: send the

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Ingo Molnar
* Priit Laes [EMAIL PROTECTED] wrote: x86: fix build on some non-C locales[1]. For some locales regex range [a-zA-Z] does not work as it is supposed to so we have to specify LANG=C to make it work as intended. thanks, applied. Ingo -- To unsubscribe from this list: send the line

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Sam Ravnborg
Hi Prit. Could we just use [[:alnum:]] and drop the setting of LC_ALL? Sam -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Priit Laes
Ühel kenal päeval, P, 2008-02-24 kell 10:03, kirjutas Sam Ravnborg: Hi Prit. Could we just use [[:alnum:]] and drop the setting of LC_ALL? Yes, it works. Reverted the previous patch and did this: diff --git a/arch/x86/vdso/Makefile b/arch/x86/vdso/Makefile index f385a4b..b8bd0c4 100644 ---

Re: [PATCH] x86: fix build on non-C locales.

2008-02-24 Thread Ingo Molnar
* Priit Laes [EMAIL PROTECTED] wrote: sed-vdsosym := -e 's/^00*/0/' \ - -e 's/^\([0-9a-fA-F]*\) . \(VDSO[a-zA-Z0-9_]*\)$$/\2 = 0x\1;/p' + -e 's/^\([[:xdigit:]]*\) . \(VDSO[[:alnum:]_]*\)$$/\2 = 0x\1;/p' thanks, i picked up this one instead of the first version. Ingo --

[PATCH] x86: fix build on non-C locales.

2008-02-23 Thread Priit Laes
x86: fix build on some non-C locales[1]. For some locales regex range [a-zA-Z] does not work as it is supposed to so we have to specify LANG=C to make it work as intended. [1] http://en.wikipedia.org/wiki/Estonian_alphabet Signed-off-by: Priit Laes <[EMAIL PROTECTED]> diff --git

[PATCH] x86: fix build on non-C locales.

2008-02-23 Thread Priit Laes
x86: fix build on some non-C locales[1]. For some locales regex range [a-zA-Z] does not work as it is supposed to so we have to specify LANG=C to make it work as intended. [1] http://en.wikipedia.org/wiki/Estonian_alphabet Signed-off-by: Priit Laes [EMAIL PROTECTED] diff --git