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

Unified Diff: build/java.gypi

Issue 13107002: [Android] Allow Java libraries to access resources from dependencies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: removed gyp bug workaround Created 7 years, 9 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/process_resources.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/java.gypi
diff --git a/build/java.gypi b/build/java.gypi
index 1ab72c20b7f706d81981e8885fbf0ae258a984e4..2bc8980eee6600846035986b30cdf35169d1d1c9 100644
--- a/build/java.gypi
+++ b/build/java.gypi
@@ -88,6 +88,7 @@
'R_stamp': '<(intermediate_dir)/resources.stamp',
'generated_src_dirs': ['<(R_dir)'],
'additional_input_paths': ['<(R_stamp)'],
+ 'additional_res_dirs': [],
},
'all_dependent_settings': {
'variables': {
@@ -133,6 +134,9 @@
'message': 'processing resources for <(_target_name)',
'variables': {
'android_manifest': '<(DEPTH)/build/android/AndroidManifest.xml',
+ # Include the depedencies' res dirs so that references to
cjhopman 2013/03/27 00:54:04 nit: s/depedencies/dependencies
newt (away) 2013/03/27 01:20:13 Done.
+ # resources in dependencies can be resolved.
+ 'all_res_dirs': ['<@(res_input_dirs)', '>@(additional_res_dirs)'],
},
'inputs': [
'<(DEPTH)/build/android/pylib/build_utils.py',
@@ -147,7 +151,7 @@
'--android-sdk', '<(android_sdk)',
'--android-sdk-tools', '<(android_sdk_tools)',
'--R-dir', '<(R_dir)',
- '--res-dirs', '<(res_input_dirs)',
+ '--res-dirs', '>(all_res_dirs)',
'--crunch-input-dir', '>(res_dir)',
'--crunch-output-dir', '<(res_crunched_dir)',
'--android-manifest', '<(android_manifest)',
« no previous file with comments | « build/android/process_resources.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698