Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(549)

Side by Side Diff: platform_tools/android/bin/android_install_app

Issue 1259123003: Update Android Testing apps to support release mode (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | platform_tools/android/bin/android_setup.sh » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/bin/bash 1 #!/bin/bash
2 # 2 #
3 # android_install_app: installs the skia sampleApp on the device. 3 # android_install_app: installs the skia sampleApp on the device.
4 4
5 function print_usage { 5 function print_usage {
6 echo "USAGE: android_install_app [options]" 6 echo "USAGE: android_install_app [options]"
7 echo " Options: -f Forces the package to be installed by removing any " 7 echo " Options: -f Forces the package to be installed by removing any "
8 echo " previously installed packages" 8 echo " previously installed packages"
9 echo " -h Prints this help message" 9 echo " -h Prints this help message"
10 echo " --release Install the release build of Skia" 10 echo " --release Install the release build of Skia"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 app="sample_app" 47 app="sample_app"
48 fi 48 fi
49 49
50 50
51 if [[ "$forceRemoval" == "true" ]]; 51 if [[ "$forceRemoval" == "true" ]];
52 then 52 then
53 echo "Forcing removal of previously installed packages" 53 echo "Forcing removal of previously installed packages"
54 $ADB ${DEVICE_SERIAL} uninstall com.skia > /dev/null 54 $ADB ${DEVICE_SERIAL} uninstall com.skia > /dev/null
55 fi 55 fi
56 56
57 BUILD_TYPE_LC=$(echo $BUILDTYPE | tr "[:upper:]" "[:lower:]") 57 if [[ "$BUILDTYPE" == "Release" ]];
borenet 2015/07/31 16:57:43 Do we care about configs like "Release_Developer"?
58 then
59 apk_suffix="release-unsigned.apk"
60 else
61 apk_suffix="debug.apk"
62 fi
63
64
58 APP_LC=$(echo $app | tr "[:upper:]" "[:lower:]") 65 APP_LC=$(echo $app | tr "[:upper:]" "[:lower:]")
59 66
60 echo "Installing ${APP_LC} from ${APP_LC}/build/outputs/apk/${APP_LC}-${ANDROID_ ARCH}-${BUILD_TYPE_LC}.apk" 67 echo "Installing ${APP_LC} from ${APP_LC}/build/outputs/apk/${APP_LC}-${ANDROID_ ARCH}-${apk_suffix}"
61 $ADB ${DEVICE_SERIAL} install -r ${SCRIPT_DIR}/../apps/${APP_LC}/build/outputs/a pk/${APP_LC}-${ANDROID_ARCH}-${BUILD_TYPE_LC}.apk 68 $ADB ${DEVICE_SERIAL} install -r ${SCRIPT_DIR}/../apps/${APP_LC}/build/outputs/a pk/${APP_LC}-${ANDROID_ARCH}-${apk_suffix}
62 69
OLDNEW
« no previous file with comments | « no previous file | platform_tools/android/bin/android_setup.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698