From 0e7f64ef60f59ecfba2ce081feb815c4d20c8979 Mon Sep 17 00:00:00 2001
From: Pat Thoyts <patthoyts@users.sourceforge.net>
Date: Fri, 11 Jun 2010 11:53:38 +0100
Subject: [PATCH] Make opticharger less verbose and clean up after itself.

The verbose output of opticharger can obscure warning messages about missing
tools so this patch reduces the noise.
The script leaves droppings in /tmp so clean this up on exit using a trap.

Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net>
---
 tools/opticharger | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/tools/opticharger b/tools/opticharger
index ecea88b2..fd77f3cf 100755
--- a/tools/opticharger
+++ b/tools/opticharger
@@ -18,7 +18,8 @@ else
 fi
 
 set -e
-
+QUIET=1
+QFLAG=-q
 BASE=`pwd`
 if [ "$APKCERTS" = "" ];
 then
@@ -69,7 +70,7 @@ then
         fi
     else
         APKINFO=`grep "name=\"$NAME\"" $APKCERTS`;
-        echo "APKINFO: $APKINFO";
+        [ $QUIET ] || echo "APKINFO: $APKINFO";
         if [ "$APKINFO" = "" ];
         then
             echo "No apk info for $NAME";
@@ -84,16 +85,17 @@ then
         fi
     fi
 
-    echo "Certificate: $CERT";
+    [ $QUIET ] || echo "Certificate: $CERT";
 
-    rm -rf /tmp/$NAME
+    [ -d /tmp/$NAME ] && rm -rf /tmp/$NAME
     mkdir /tmp/$NAME
+    trap "rm -rf /tmp/$NAME; exit" INT TERM EXIT
     cd /tmp/$NAME
-    unzip $BASE/$1
+    unzip -q $BASE/$1
     for x in `find . -name "*.png" | grep -v "\.9.png$" | tr "\n" " "`
     do
-        echo "Crushing $x"
-        pngcrush $x $x.crushed 1>/dev/null
+        [ $QUIET ] || echo "Crushing $x"
+        pngcrush $QFLAG $x $x.crushed 1>/dev/null
         if [ -e "$x.crushed" ];
         then
             mv $x.crushed $x
@@ -101,17 +103,19 @@ then
     done
     cp $BASE/$1 $BASE/$1.old
 
-    echo "Repacking apk.."
+    [ $QUIET ] || echo "Repacking apk.."
     aapt p -0 res/raw -0 res/raw-en -F $NAME .
     
-        echo "Resigning with cert: `echo $CERT`"
+    [ $QUIET ] || echo "Resigning with cert: `echo $CERT`"
 
-    echo java -jar $ANDROID_BUILD_TOP/out/host/$OUT_TARGET_HOST/framework/signapk.jar $ANDROID_BUILD_TOP/$CERT $ANDROID_BUILD_TOP/$KEY $NAME signed_$NAME
+    [ $QUIET ] || echo java -jar $ANDROID_BUILD_TOP/out/host/$OUT_TARGET_HOST/framework/signapk.jar $ANDROID_BUILD_TOP/$CERT $ANDROID_BUILD_TOP/$KEY $NAME signed_$NAME
     java -jar $ANDROID_BUILD_TOP/out/host/$OUT_TARGET_HOST/framework/signapk.jar $ANDROID_BUILD_TOP/$CERT $ANDROID_BUILD_TOP/$KEY $NAME signed_$NAME
-    echo "Zipalign.."
+    [ $QUIET ] || echo "Zipalign.."
     zipalign -f 4 signed_$NAME $BASE/$1
-    ls -l $BASE/$1.old
-    ls -l $BASE/$1
+    if [ ! $QUIET ]; then
+        ls -l $BASE/$1.old
+        ls -l $BASE/$1
+    fi
     rm $BASE/$1.old
 else
     echo "Usage: $0 [apk file]"