Re: [PATCH] Remove bashisms from scripts/extract-ikconfig

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Christian Kujau <lists@...>
Cc: Willy Tarreau <w@...>, Julio M. Merino Vidal <jmerino@...>, <linux-kernel@...>
Date: Wednesday, November 14, 2007 - 7:44 pm

Hi Christian,

Does the patch below satisfy your needs here?

Thanks,
~Randy

---

From: Christian Kujau <lists@nerdbynature.de>

Remove bashisms from scripts/extract-ikconfig (function and
typeset words).

Tested under dash v0.5.4 (latest/current).

Signed-off-by: Christian Kujau <lists@nerdbynature.de>
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
---
 scripts/extract-ikconfig |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

--- linux-2.6.24-rc2-git2.orig/scripts/extract-ikconfig
+++ linux-2.6.24-rc2-git2/scripts/extract-ikconfig
@@ -8,8 +8,9 @@ test -e $binoffset || cc -o $binoffset .
 
 IKCFG_ST="0x49 0x4b 0x43 0x46 0x47 0x5f 0x53 0x54"
 IKCFG_ED="0x49 0x4b 0x43 0x46 0x47 0x5f 0x45 0x44"
-function dump_config {
-    typeset file="$1"
+
+dump_config() {
+    file="$1"
 
     start=`$binoffset $file $IKCFG_ST 2>/dev/null`
     [ "$?" != "0" ] && start="-1"
@@ -27,7 +28,6 @@ function dump_config {
     exit 0
 }
 
-
 usage()
 {
 	echo "  usage: extract-ikconfig [b]zImage_filename"

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Remove bashisms from scripts/extract-ikconfig, Christian Kujau, (Sat Jun 9, 3:22 pm)
Re: [PATCH] Remove bashisms from scripts/extract-ikconfig, Julio M. Merino Vidal, (Sat Jun 9, 3:26 pm)
Re: [PATCH] Remove bashisms from scripts/extract-ikconfig, Christian Kujau, (Sat Jun 9, 3:34 pm)
Re: [PATCH] Remove bashisms from scripts/extract-ikconfig, Christian Kujau, (Sat Jun 9, 4:14 pm)
Re: [PATCH] Remove bashisms from scripts/extract-ikconfig, Randy Dunlap, (Wed Nov 14, 7:44 pm)
Re: [PATCH] Remove bashisms from scripts/extract-ikconfig, Julio M. Merino Vidal, (Sat Jun 9, 3:39 pm)