Tests like [ -z $foobar ] are not portable.  The variable needs to
be quoted if it might be unset, like [ -z "$foobar" ].  I've included
a patch to the abiword script-generation scripts below.

Thanks!

-- 
- Kevin Vajk
  <[EMAIL PROTECTED]>


Index: abi/src/pkg/common/unix/data/tbz_install.sh
===================================================================
RCS file: /cvsroot/abi/src/pkg/common/unix/data/tbz_install.sh,v
retrieving revision 1.2
diff -u -u -r1.2 tbz_install.sh
--- abi/src/pkg/common/unix/data/tbz_install.sh 2000/10/10 00:39:30     1.2
+++ abi/src/pkg/common/unix/data/tbz_install.sh 2001/03/02 01:25:23
@@ -186,19 +186,19 @@
 #now try to guess locale
 locale=\$LC_ALL        #it's incorrect to set this variable, but someone
                #might set it incorrectly.
-if [ -z \$locale ]
+if [ -z "\$locale" ]
 then
     locale=\$LANG
 fi

-if [ ! -z \$locale ]
+if [ ! -z "\$locale" ]
 then
     #now guess encoding
     encoding=\`echo \$locale | sed -e 's/^.*\.\(.*\)\$/\1/'\`
-    if [ ! -z \$encoding ]
+    if [ ! -z "\$encoding" ]
     then
        addfontdir=\$ABISUITE_FONT_HOME/\$encoding
-       if [ ! -z \$addfontdir ]
+       if [ ! -z "\$addfontdir" ]
        then
            if [ -d \$addfontdir ]
            then
Index: abi/src/pkg/common/unix/data/tgz_install.sh
===================================================================
RCS file: /cvsroot/abi/src/pkg/common/unix/data/tgz_install.sh,v
retrieving revision 1.6
diff -u -u -r1.6 tgz_install.sh
--- abi/src/pkg/common/unix/data/tgz_install.sh 2000/10/10 00:39:30     1.6
+++ abi/src/pkg/common/unix/data/tgz_install.sh 2001/03/02 01:25:24
@@ -186,19 +186,19 @@
 #now try to guess locale
 locale=\$LC_ALL        #it's incorrect to set this variable, but someone
                #might set it incorrectly.
-if [ -z \$locale ]
+if [ -z "\$locale" ]
 then
     locale=\$LANG
 fi

-if [ ! -z \$locale ]
+if [ ! -z "\$locale" ]
 then
     #now guess encoding
     encoding=\`echo \$locale | sed -e 's/^.*\.\(.*\)\$/\1/'\`
-    if [ ! -z \$encoding ]
+    if [ ! -z "\$encoding" ]
     then
        addfontdir=\$ABISUITE_FONT_HOME/\$encoding
-       if [ ! -z \$addfontdir ]
+       if [ ! -z "\$addfontdir" ]
        then
            if [ -d \$addfontdir ]
            then
Index: abi/src/pkg/common/unix/scripts/makewrapper.sh
===================================================================
RCS file: /cvsroot/abi/src/pkg/common/unix/scripts/makewrapper.sh,v
retrieving revision 1.9
diff -u -u -r1.9 makewrapper.sh
--- abi/src/pkg/common/unix/scripts/makewrapper.sh      2000/10/10 00:39:31     1.9
+++ abi/src/pkg/common/unix/scripts/makewrapper.sh      2001/03/02 01:25:24
@@ -61,19 +61,19 @@
 #now try to guess locale
 locale=\$LC_ALL        #it's incorrect to set this variable, but someone
                #might set it incorrectly.
-if [ -z \$locale ]
+if [ -z "\$locale" ]
 then
     locale=\$LANG
 fi

-if [ ! -z \$locale ]
+if [ ! -z "\$locale" ]
 then
     #now guess encoding
     encoding=\`echo \$locale | sed -e 's/^.*\.\(.*\)\$/\1/'\`
-    if [ ! -z \$encoding ]
+    if [ ! -z "\$encoding" ]
     then
        addfontdir=\$ABISUITE_FONT_HOME/\$encoding
-       if [ ! -z \$addfontdir ]
+       if [ ! -z "\$addfontdir" ]
        then
            if [ -d \$addfontdir ]
            then
@@ -107,7 +107,7 @@
     exit
 fi
 #Check to make sure we don't stomp on anything
-if [ -z \$currentFonts ]
+if [ -z "\$currentFonts" ]
 then
     # Set post run-time font path
     if [ -d \$ABISUITE_FONT_HOME ]



Reply via email to