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

Unified Diff: build/java_apk.gypi

Issue 14017010: [Android] Use a script on the device for creating library symlinks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 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/android/gyp/create_device_library_links.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/java_apk.gypi
diff --git a/build/java_apk.gypi b/build/java_apk.gypi
index ecf5474271f206d8dcc8b324981fc361da625ba6..5fbdd48b7fd0870657a703a968f9126580b9a2e5 100644
--- a/build/java_apk.gypi
+++ b/build/java_apk.gypi
@@ -106,8 +106,10 @@
'unsigned_apk_path': '<(intermediate_dir)/<(apk_name)-unsigned.apk',
'final_apk_path%': '<(PRODUCT_DIR)/apks/<(apk_name).apk',
'source_dir': '<(java_in_dir)/src',
- 'apk_install_stamp': '<(intermediate_dir)/apk_install.stamp',
+ 'apk_install_record': '<(intermediate_dir)/apk_install.record.stamp',
'apk_package_native_libs_dir': '<(intermediate_dir)/libs',
+ 'symlink_script_host_path': '<(intermediate_dir)/create_symlinks.sh',
+ 'symlink_script_device_path': '/data/local/tmp/chromium/<(_target_name)/create_symlinks.sh',
},
# Pass the jar path to the apk's "fake" jar target. This would be better as
# direct_dependent_settings, but a variable set by a direct_dependent_settings
@@ -210,22 +212,24 @@
'includes': ['../build/android/push_libraries.gypi'],
},
{
- 'action_name': 'create_library_links',
+ 'action_name': 'create device library symlinks',
'message': 'Creating links on device for <(_target_name).',
'inputs': [
+ '<(DEPTH)/build/android/gyp/util/build_utils.py',
'<(DEPTH)/build/android/gyp/create_device_library_links.py',
- '<(apk_install_stamp)',
- '<(push_stamp)'
+ '<(apk_install_record)',
+ '<(ordered_libraries_file)',
],
'outputs': [
'<(link_stamp)'
],
'action': [
'python', '<(DEPTH)/build/android/gyp/create_device_library_links.py',
- '--apk=<(final_apk_path)',
'--libraries-json=<(ordered_libraries_file)',
- '--libraries-dir=<(libraries_source_dir)',
+ '--script-host-path=<(symlink_script_host_path)',
+ '--script-device-path=<(symlink_script_device_path)',
'--target-dir=<(device_library_dir)',
+ '--apk=<(final_apk_path)',
'--stamp=<(link_stamp)',
],
},
@@ -273,18 +277,18 @@
'<(final_apk_path)',
],
'outputs': [
- '<(apk_install_stamp)',
+ '<(apk_install_record)',
# If a user switches the connected device, the APK may need to be
# installed even if there have been no changes. To ensure that the
# APK on the device is always up-to-date, this step should always
# be triggered.
- '<(apk_install_stamp).fake',
+ '<(apk_install_record).fake',
],
'action': [
'python', '<(DEPTH)/build/android/gyp/apk_install.py',
'--android-sdk-tools=<(android_sdk_tools)',
'--apk-path=<(final_apk_path)',
- '--stamp=<(apk_install_stamp)'
+ '--install-record=<(apk_install_record)'
],
},
],
« no previous file with comments | « build/android/gyp/create_device_library_links.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698