diff options
author | mtklein@google.com <mtklein@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-09-30 17:32:09 +0000 |
---|---|---|
committer | mtklein@google.com <mtklein@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-09-30 17:32:09 +0000 |
commit | 48f9e1f2c1a164128fa3e1fc5e264aab16b2b884 (patch) | |
tree | 2bba6b16d3156aa1bf3ba8cdb9a4c6939973c9d5 /platform_tools/android/bin | |
parent | 741ea9e0098bde6bdfc71c65572b51ce9596944a (diff) |
Add android_ninja, like android_make for ninja.
android_ninja {tests,bench,gm,bench_pictures,render_pictures} all build and run
fine. They build so fast I thought maybe they were broken.
Note that android_ninja with no argument is failing:
ninja: Entering directory `out/config/android-arm_v7_thumb/Debug'
ninja: error: 'lib.target/libSampleApp.so', needed by 'android/libs/armeabi-v7a/libSampleApp.so', missing and no known rule to make it
Will look into this.
BUG=
R=djsollen@google.com
Committed: https://code.google.com/p/skia/source/detail?r=11525
Review URL: https://codereview.chromium.org/24833002
git-svn-id: http://skia.googlecode.com/svn/trunk@11527 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'platform_tools/android/bin')
-rwxr-xr-x | platform_tools/android/bin/android_make | 54 | ||||
l--------- | platform_tools/android/bin/android_ninja | 1 | ||||
-rwxr-xr-x | platform_tools/android/bin/android_setup.sh | 4 |
3 files changed, 19 insertions, 40 deletions
diff --git a/platform_tools/android/bin/android_make b/platform_tools/android/bin/android_make index 10ddd3301b..9696dbef55 100755 --- a/platform_tools/android/bin/android_make +++ b/platform_tools/android/bin/android_make @@ -1,46 +1,26 @@ #!/bin/bash -SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )" +# Fail-fast if anything in the script fails. +set -e -# remove the existing .android_config file prior to running android_setup. If -# we did not remove this here then we would build for whatever device type was +# Remove any existing .android_config file before running android_setup. If we +# did not remove this now then we would build for whatever device type was # listed in the .android_config instead of the default device type. -if [ -f .android_config ] -then - rm .android_config -fi +rm -f .android_config -# run the config to setup the environment +SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )" source $SCRIPT_DIR/android_setup.sh -# write the device id into the .android_config file -echo $DEVICE_ID > .android_config - -for arg in ${APP_ARGS[@]} -do - if [[ "${arg}" == "--use-ccache" ]]; - then - if [[ -z "$ANDROID_MAKE_CCACHE" ]]; - then - ANDROID_MAKE_CCACHE=$(which ccache) - fi - else - makeVars=("${makeVars[@]}" "${arg}") - fi - -shift -done - -if [[ -n "$ANDROID_MAKE_CCACHE" ]]; then - $ANDROID_MAKE_CCACHE --version &> /dev/null - if [[ "$?" != "0" ]]; then - echo "Unable to find ccache!" - exit 1 - fi +if [ $(basename $0) = "android_make" ]; then + GYP_GENERATORS=make-android make $APP_ARGS +else + GYP_GENERATORS=ninja ./gyp_skia + OUT=$SKIA_OUT/${BUILDTYPE-Debug} # Defaults to Debug if BUILDTYPE isn't set. + ninja -C $OUT $APP_ARGS + ln -sf lib $OUT/lib.target # android_run_skia looks in lib.target; ninja writes to lib. fi -make ${makeVars[@]} -if [ $? != 0 ] -then - exit 1; -fi +# Write the device id into the .android_config file. This tells +# android_run_skia the last build we completed. +echo $DEVICE_ID > .android_config + diff --git a/platform_tools/android/bin/android_ninja b/platform_tools/android/bin/android_ninja new file mode 120000 index 0000000000..68a0fb120e --- /dev/null +++ b/platform_tools/android/bin/android_ninja @@ -0,0 +1 @@ +android_make
\ No newline at end of file diff --git a/platform_tools/android/bin/android_setup.sh b/platform_tools/android/bin/android_setup.sh index 7c3ba6a243..0629c6a6e8 100755 --- a/platform_tools/android/bin/android_setup.sh +++ b/platform_tools/android/bin/android_setup.sh @@ -165,7 +165,7 @@ setup_device() { TARGET_DEVICE=$(cat .android_config) echo "INFO: no target device was specified so using the device (${TARGET_DEVICE}) from the most recent build" else - TARGET_DEVICE="arm_v7_thumb" + TARGET_DEVICE="arm_v7_thumb" echo "INFO: no target device type was specified so using the default '${TARGET_DEVICE}'" fi fi @@ -237,8 +237,6 @@ setup_device() { fi DEFINES="${DEFINES} android_toolchain=${TOOLCHAIN_TYPE}" - # Use the "android" flavor of the Makefile generator for both Linux and OS X. - exportVar GYP_GENERATORS "make-android" exportVar GYP_DEFINES "$DEFINES" exportVar SKIA_OUT "out/config/android-${TARGET_DEVICE}" } |