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

Unified Diff: content/content_shell.gypi

Issue 10914130: Make apks be built in a consistent way (Closed) Base URL: http://git.chromium.org/chromium/src.git@ant_ijpforrealz
Patch Set: Rebase Created 8 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/java_apk.gypi ('k') | content/content_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/content_shell.gypi
diff --git a/content/content_shell.gypi b/content/content_shell.gypi
index f48b06880a28dd21593b026f048644ca54a4e910..7ce35d555fabd06f1327e2cd10edd12f2cec4a71 100644
--- a/content/content_shell.gypi
+++ b/content/content_shell.gypi
@@ -602,6 +602,14 @@
'../media/media.gyp:media_java',
'../net/net.gyp:net_java',
],
+ 'variables': {
+ 'package_name': 'content_shell',
+ 'apk_name': 'ContentShell',
+ 'java_in_dir': 'shell/android/java',
+ 'resource_dir': 'shell/android/res',
+ 'native_libs_paths': ['<(PRODUCT_DIR)/content_shell/libs/<(android_app_abi)/libcontent_shell_content_view.so'],
+ 'additional_input_paths': ['<(PRODUCT_DIR)/content_shell/assets/content_shell.pak'],
+ },
'actions': [
{
'action_name': 'copy_and_strip_so',
@@ -615,45 +623,8 @@
'<@(_outputs)',
],
},
- {
- 'action_name': 'content_shell_apk',
- 'inputs': [
- 'shell/android/java/content_shell_apk.xml',
- 'shell/android/java/AndroidManifest.xml',
- '../build/android/ant/common.xml',
- '../build/android/ant/sdk-targets.xml',
- '<!@(find shell/android/java -name "*.java")',
- '<!@(find shell/android/res -name "*")',
- '>@(input_jars_paths)',
- '<(PRODUCT_DIR)/content_shell/assets/content_shell.pak',
- '<(PRODUCT_DIR)/content_shell/libs/<(android_app_abi)/libcontent_shell_content_view.so',
- ],
- 'outputs': [
- # Awkwardly, we build a Debug APK even when gyp is in
- # Release mode. I don't think it matters (e.g. we're
- # probably happy to not codesign) but naming should be
- # fixed.
- '<(PRODUCT_DIR)/content_shell/ContentShell-debug.apk',
- ],
- 'action': [
- # Pass the build type to ant. Currently it only assumes
- # debug mode in java. Release mode will break the current
- # workflow.
- 'ant',
- '-DPRODUCT_DIR=<(ant_build_out)',
- '-DAPP_ABI=<(android_app_abi)',
- '-DANDROID_SDK=<(android_sdk)',
- '-DANDROID_SDK_ROOT=<(android_sdk_root)',
- '-DANDROID_SDK_TOOLS=<(android_sdk_tools)',
- '-DANDROID_SDK_VERSION=<(android_sdk_version)',
- '-DANDROID_GDBSERVER=<(android_gdbserver)',
- '-DCONFIGURATION_NAME=<(CONFIGURATION_NAME)',
- '-DINPUT_JARS_PATHS=>(input_jars_paths)',
- '-buildfile',
- 'shell/android/java/content_shell_apk.xml',
- ],
- }
],
+ 'includes': [ '../build/java_apk.gypi' ],
},
],
}], # OS=="android"
« no previous file with comments | « build/java_apk.gypi ('k') | content/content_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698