Bug#415119: dash compiles incorrectly when LC_ALL isn't C

2007-03-23 Thread Roy Marples
On Fri, 23 Mar 2007 10:47:44 +
Gerrit Pape [EMAIL PROTECTED] wrote:

 tags 415119 + patch
 forwarded 415119 upstream
 quit
 
 On Fri, Mar 16, 2007 at 08:02:16AM +, Roy Marples wrote:
  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 for the patch, Roy, I forwarded it upstream
 
 Regards, Gerrit.

I thought Debian was upstream - based on the name of the package. Heh.
Do you have a URL for upstream so I can update our ebuild?

Thanks

Roy


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



Bug#415119: dash compiles incorrectly when LC_ALL isn't C

2007-03-23 Thread Gerrit Pape
On Fri, Mar 23, 2007 at 10:57:20AM +, Roy Marples wrote:
 On Fri, 23 Mar 2007 10:47:44 +
 Gerrit Pape [EMAIL PROTECTED] wrote:
  Thanks for the patch, Roy, I forwarded it upstream

 I thought Debian was upstream - based on the name of the package. Heh.

It's been so before, Herbert Xu is upstream, but left Debian some time
ago.

 Do you have a URL for upstream so I can update our ebuild?

Sure, http://gondor.apana.org.au/~herbert/dash/

Regards, Gerrit.


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



Bug#415119: dash compiles incorrectly when LC_ALL isn't C

2007-03-16 Thread Roy Marples
Package: dash
Version 0.5.3-7

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--- dash-0.5.3.orig/src/mkbuiltins	2005-11-26 03:17:55.0 +
+++ dash-0.5.3/src/mkbuiltins	2007-03-15 21:23:51.448422603 +
@@ -65,7 +65,7 @@
 		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)