[coreboot] msrtool: posix-ify configure script

Andriy Gapon avg at icyb.net.ua
Thu Nov 26 13:32:52 CET 2009


Please review the following patch.
It changes a few bash-specific constructs to more portable syntax specified by
POSIX.  After the change the script keeps working with bash interpreter and can
also be interpreted by FreeBSD /bin/sh.

Signed-off-by:	Andriy Gapon <avg at icyb.net.ua>

Index: configure
===================================================================
--- configure	(revision 4776)
+++ configure	(working copy)
@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/bin/sh
 #
 # This file is part of msrtool.
 #
@@ -25,7 +25,7 @@
 REV="`svnversion -c . 2>/dev/null | sed 's,.*:,,' 2>/dev/null`"
 VERSION="${VERSION:-$REV}"

-function findprog {
+findprog() {
 	NPARMS=$#
 	WHAT="${1}"
 	shift
@@ -41,7 +41,7 @@
 	while test $i -lt $NPARMS; do
 		test -z "${1}" && {
 			shift
-			i=$[$i+1]
+			i=$(($i+1))
 			continue
 		}
 		FILE="`which "${1}" 2>/dev/null`"
@@ -50,7 +50,7 @@
 			break
 		}
 		shift
-		i=$[$i+1]
+		i=$(($i+1))
 	done
 	test -z "${1}" && {
 		echo "  not found!" 1>&2
@@ -62,7 +62,7 @@
 	exit 0
 }

-function trycompile {
+trycompile() {
 	NPARMS=$#
 	WHAT="${1}"
 	shift
@@ -84,7 +84,7 @@
 			exit 0
 		}
 		shift
-		i=$[$i+1]
+		i=$(($i+1))
 	done
 	echo "failed!" 1>&2
 	echo 1>&2
@@ -99,7 +99,7 @@
 	exit 1
 }

-function trylink {
+trylink() {
 	NPARMS=$#
 	WHAT="${1}"
 	shift
@@ -121,7 +121,7 @@
 			exit 0
 		}
 		shift
-		i=$[$i+1]
+		i=$(($i+1))
 	done
 	echo "failed!" 1>&2
 	echo 1>&2

-- 
Andriy Gapon




More information about the coreboot mailing list