commit 2696e5a0676c1ba39ee7f97b48ef65f5dc986d4f
author Rick Harris <rconradharris@gmail.com> Mon Aug 18 13:53:42 2014 -0500
committer Rick Harris <rconradharris@gmail.com> Mon Aug 18 13:55:23 2014 -0500
tree cbff64cc66f640e8f213815afbf4574c84b86de2
parent a6a45467c412476b2cddb8526a97e421b0b74db7

Guard for worlddump in case LOGDIR is undefined

If LOGDIR is undefined, we'll end up calling worldump with the `-d` option but
pass in an empty string as the option value. This causes an argument parsing
error inside worlddump.

The proposed fix is to detect if LOGDIR is undefined, and if so, omit the `-d`
argument.

Change-Id: I77968849ebd3e3bebd2e5929b87b60fc1b6541bf
Closes-Bug: 1333321

stack.sh

diff --git a/stack.sh b/stack.sh
index 7a7655c..9b9c245 100755
--- a/stack.sh
+++ b/stack.sh
@@ -631,7 +631,11 @@
 
     if [[ $r -ne 0 ]]; then
         echo "Error on exit"
-        ./tools/worlddump.py -d $LOGDIR
+        if [[ -z $LOGDIR ]]; then
+            ./tools/worlddump.py
+        else
+            ./tools/worlddump.py -d $LOGDIR
+        fi
     fi
 
     exit $r