Mojo: mojob.sh: Make --release (etc.) work properly again.

mojo/tools/mojob.sh

Index: mojo/tools/mojob.sh
diff --git a/mojo/tools/mojob.sh b/mojo/tools/mojob.sh
index 7092c791f696995a5e2109b721a7f6a375404b56..165da0f877a41d7bbdf23ad80cebf839f772ee76 100755
--- a/mojo/tools/mojob.sh
+++ b/mojo/tools/mojob.sh
@@ -101,6 +101,17 @@ COMPILER=clang
 GOMA=auto
 make_gn_args() {
   local args=()
+  # TODO(vtl): It's a bit of a hack to infer the build type from the output
+  # directory name, but it's what we have right now (since we support "debug and
+  # release" mode).
+  case "$1" in
+    Debug)

[jamesr, 2014/10/21 00:46:35]

i would leave this out - is_debug=true is the default, no need to pollute args

[viettrungluu, 2014/10/21 01:04:22]

My thinking is that if you make an explicit choice (e.g., debug vs release),
then you should put in the arg, since defaults are subject to change (and this
would silently start to do the wrong thing).

(An alternative is to compare the value vs the default and set it only if it's
not the default, but that's a lot of work.)

+      args+=("is_debug=true")
+      ;;
+    Release)
+      args+=("is_debug=false")
+      ;;
+  esac
   case "$COMPILER" in
     clang)
       args+=("is_clang=true")