tools/ avoid a bashism

Austin English austinenglish at
Tue Jul 20 14:44:38 CDT 2010

On Tue, Jul 20, 2010 at 2:37 PM, Austin English <austinenglish at> wrote:
> While it's not technically a bashism, since the variable is being
> assigned by the script, it makes the LSB App Checker slightly happier.
> Also fixes the case where OLDDIR contained spaces.

Seems the patch was wrapped...

-------------- next part --------------
diff --git a/tools/ b/tools/
index 91ca600..989f5a7 100755
--- a/tools/
+++ b/tools/
@@ -35,7 +35,7 @@ TARGET=/usr/X11R6/lib/X11/fonts/misc;
 usage () {
     echo "usage: "`basename $0`" [-q] [-c charset] [-t fontdir] [-b bdftopcf] [-f fnt2bdf]"
@@ -117,7 +117,7 @@ for i in *.bdf; do
     mv "${i%.???}.pcf.gz" $TARGET 2>/dev/null
     if [ $? -ne 0 ]; then
 	$Q echo "Can't install fonts to $TARGET. Try again as the root user.";
-        $Q echo "Cleaning up..."; cd "$OLDPWD"; rm -rf "$TMPDIR"; exit 1;
+        $Q echo "Cleaning up..."; cd "$OLDDIR"; rm -rf "$TMPDIR"; exit 1;
     rm "$i";

More information about the wine-patches mailing list