Bug#415119: [PATCH] set LC_ALL=C for sort to ensure lexical order of builtincmd structure

2007-04-28 Thread Herbert Xu
On Fri, Mar 23, 2007 at 10:46:13AM +, Gerrit Pape wrote:
 Patch by Roy Marples, see http://bugs.debian.org/415119
 
 When LC_ALL is set to something like en_GB.UTF-8 then dash makes the
 builtincmd structure in the wrong lexical order, breaking bsearch,
 specifically the truecmd :

Thanks Gerrit.  I've applied a similar patch to my tree.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#415119: [PATCH] set LC_ALL=C for sort to ensure lexical order of builtincmd structure

2007-03-23 Thread Gerrit Pape
Patch by Roy Marples, see http://bugs.debian.org/415119

When LC_ALL is set to something like en_GB.UTF-8 then dash makes the
builtincmd structure in the wrong lexical order, breaking bsearch,
specifically the truecmd :

$ :
dash: :: not found

The attached patch addresses the issue.

Thanks

Roy Marples
---
 src/mkbuiltins |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/mkbuiltins b/src/mkbuiltins
index f3f91c5..91fe6b8 100644
--- a/src/mkbuiltins
+++ b/src/mkbuiltins
@@ -65,7 +65,7 @@ awk '{for (i = 2 ; i = NF ; i++) {
if ($i ~ /^-/)
line = $(++i) \t line
print line
-   }}' $temp | sort -k 1,1 | tee $temp2 | awk '{
+   }}' $temp | LC_ALL=C sort -k 1,1 | tee $temp2 | awk '{
opt = 
if (NF  2) {
opt = substr($2, 2)
-- 
1.5.0.5



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]