Add Makefile targets for running tests on Android.

tools/android-sync.sh

 #!/bin/bash
 # Copyright 2012 the V8 project authors. All rights reserved.
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 #       notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 #       copyright notice, this list of conditions and the following
 #       disclaimer in the documentation and/or other materials provided
 #       with the distribution.
 #     * Neither the name of Google Inc. nor the names of its
 #       contributors may be used to endorse or promote products derived
 #       from this software without specific prior written permission.
 #
 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-# Inspired by and based on:
-# http://src.chromium.org/viewvc/chrome/trunk/src/tools/bash-completion
+# This script pushes android binaries and test data to the device.
+# The first argument can be either "android.release" or "android.debug".
+# The second argument is a relative path to the output directory with binaries.
+# The third argument is the absolute path to the V8 directory on the host.
+# The fourth argument is the absolute path to the V8 directory on the device.
 
-# Flag completion rule for bash.
-# To load in your shell, "source path/to/this/file".
+ARCH_MODE=$1
+OUTDIR=$2
+HOST_V8=$3
+ANDROID_V8=$4

[Jakob Kummerow, 2012/07/02 14:25:58]

How about a check that all arguments are present, and otherwise print usage
instructions and exit?

To save you the trouble of looking it up, here's a snippet:
if [ ${#@} -lt 4 ] ; then
  echo "less than 4 arguments"
fi

[ulan, 2012/07/02 16:18:23]

Done.

 
-v8_source=$(readlink -f $(dirname $BASH_SOURCE)/..)
-
-_v8_flag() {
-  local cur defines targets
-  cur="${COMP_WORDS[COMP_CWORD]}"
-  defines=$(cat src/flag-definitions.h \
-    | grep "^DEFINE" \
-    | grep -v "DEFINE_implication" \
-    | sed -e 's/_/-/g')
-  targets=$(echo "$defines" \
-    | sed -ne 's/^DEFINE-[^(]*(\([^,]*\).*/--\1/p'; \
-    echo "$defines" \
-    | sed -ne 's/^DEFINE-bool(\([^,]*\).*/--no\1/p'; \
-    cat src/d8.cc \
-    | grep "strcmp(argv\[i\]" \
-    | sed -ne 's/^[^"]*"--\([^"]*\)".*/--\1/p')
-  COMPREPLY=($(compgen -W "$targets" -- "$cur"))
-  return 0
+function sync_file {
+  local FILE=$1
+  local ANDROID_HASH=`adb shell "md5 $ANDROID_V8/$FILE"`

[Jakob Kummerow, 2012/07/02 14:25:58]

Please use $(...) instead of `...`

[ulan, 2012/07/02 16:18:23]

Done.

+  local HOST_HASH=`md5sum $HOST_V8/$FILE`

[Jakob Kummerow, 2012/07/02 14:25:58]

again $(...)

And if you don't want this to fail horribly in case $HOST_V8 ever contains
spaces, please put quotes around all variables containing user-defined paths.

[ulan, 2012/07/02 16:18:23]

Done.

+  if [ "${ANDROID_HASH%% *}" != "${HOST_HASH%% *}" ]
+  then

[Jakob Kummerow, 2012/07/02 14:25:58]

nit: you can put the "then" on the "if"'s line (like we do for '{') by inserting
a ';' between them:

if [ condition ] ; then

[ulan, 2012/07/02 16:18:23]

Done.

+    adb push $HOST_V8/$FILE $ANDROID_V8/$FILE &> /dev/null
+  fi
 }
 
-complete -F _v8_flag -f d8
+function sync_dir {
+  local DIR=$1
+  echo -n "sync to $ANDROID_V8/$DIR"
+  for FILE in `find $HOST_V8/$DIR -type f`

[Jakob Kummerow, 2012/07/02 14:25:58]

again $(...)

[ulan, 2012/07/02 16:18:23]

Done.

+  do

[Jakob Kummerow, 2012/07/02 14:25:58]

nit: again, for consistency with '{', I suggest removing the line break:

for foo ; do

[ulan, 2012/07/02 16:18:23]

Done.

+    local RELATIVE_FILE=${FILE:${#HOST_V8}}
+    sync_file $RELATIVE_FILE
+    echo -n "."

[Jakob Kummerow, 2012/07/02 14:25:58]

How about moving this line into sync_file and removing it everywhere else?

[ulan, 2012/07/02 16:18:23]

Done.

+  done
+  echo ""
+}
+
+echo -n "sync to $ANDROID_V8/$OUTDIR/$ARCH_MODE"
+sync_file $OUTDIR/$ARCH_MODE/cctest
+echo -n "."
+sync_file $OUTDIR/$ARCH_MODE/d8
+echo -n "."
+sync_file $OUTDIR/$ARCH_MODE/preparser
+echo -n "."
+sync_file $OUTDIR/$ARCH_MODE/process

[Jakob Kummerow, 2012/07/02 14:25:58]

I don't think the "process" binary is required for anything.

[ulan, 2012/07/02 16:18:23]

Done.

+echo -n "."
+sync_file $OUTDIR/$ARCH_MODE/shell

[Jakob Kummerow, 2012/07/02 14:25:58]

"shell" is not required.

[ulan, 2012/07/02 16:18:23]

Done.

+echo "."
+echo -n "sync to $ANDROID_V8/tools"
+sync_file tools/consarray.js
+echo -n "."
+sync_file tools/codemap.js
+echo -n "."
+sync_file tools/csvparser.js
+echo -n "."
+sync_file tools/profile.js
+echo -n "."
+sync_file tools/splaytree.js
+echo "."
+sync_dir test/mjsunit
+sync_dir test/preparser
+sync_dir test/stress

[Jakob Kummerow, 2012/07/02 14:25:58]

I don't think test/stress is part of a regular V8 checkout.

[ulan, 2012/07/02 16:18:23]

Done.