Stefan Allius writes:

1. Under Solaris the test command with the 'string' argument don't
   work well. So I added the '-n' argument, which might work on all
   platforms.

2. The sed under Solaris don't support the '-s' argument, which is a
   GNU extension. I changed it to '-e' which works fine for me.

1 files changed, 4 insertions, 4 deletions

diff --git a/extra/scripts/fix_includes.sh b/extra/scripts/fix_includes.sh
index 27ada1f4d..6e8c00c40 100755
--- a/extra/scripts/fix_includes.sh
+++ b/extra/scripts/fix_includes.sh
@@ -49,11 +49,11 @@ usage () {
 }
 
 HAS_MMU="y";
-while [ "$1" ]; do
+while [ -n "$1" ]; do
     case $1 in
-	-k ) shift; if [ "$1" ]; then KERNEL_SOURCE=$1; shift; else usage; fi; ;;
-	-t ) shift; if [ "$1" ]; then TARGET_ARCH=$1; shift; else usage; fi; ;;
-	-n ) shift; if [ "$1" ]; then HAS_MMU="n"; shift; else usage; fi; ;;
+	-k ) shift; if [ -n "$1" ]; then KERNEL_SOURCE=$1; shift; else usage; fi; ;;
+	-t ) shift; if [ -n "$1" ]; then TARGET_ARCH=$1; shift; else usage; fi; ;;
+	-n ) shift; if [ -n "$1" ]; then HAS_MMU="n"; shift; else usage; fi; ;;
 	-* ) usage; ;;
 	* ) usage; ;;
     esac;