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

Unified Diff: build/java.gypi

Issue 199993003: android: Remove last md5sum call from java.gypi. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/gyp/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 d93f967c49b0e8136b4a3732a7951d93d2d57440..015de7556dcfe5acb7427dcdd677c50291ce6c2d 100644
--- a/build/java.gypi
+++ b/build/java.gypi
@@ -183,16 +183,16 @@
# resources in dependencies can be resolved.
'all_res_dirs': ['<@(res_input_dirs)',
'>@(dependencies_res_input_dirs)',],
- # Write the inputs list to a file, so that the action command
- # line won't exceed the OS limits when calculating the checksum
- # of the list.
- 'inputs_list_file': '>|(inputs_list.<(_target_name).gypcmd >@(_inputs))'
+ # Write the inputs list to a file, so that its mtime is updated when
+ # the list of inputs changes.
+ 'inputs_list_file': '>|(inputs_list.<(_target_name).gypcmd >@(resource_input_paths) @>(dependencies_res_files))'
},
'inputs': [
'<(DEPTH)/build/android/gyp/util/build_utils.py',
'<(DEPTH)/build/android/gyp/process_resources.py',
'>@(resource_input_paths)',
'>@(dependencies_res_files)',
+ '>(inputs_list_file)',
],
'outputs': [
'<(R_stamp)',
@@ -209,11 +209,6 @@
'--non-constant-id',
'--custom-package', '<(R_package)',
'--stamp', '<(R_stamp)',
-
- # Add hash of inputs to the command line, so if inputs change
- # (e.g. if a resource if removed), the command will be re-run.
- # TODO(newt): remove this once crbug.com/177552 is fixed in ninja.
- '--ignore=>!(md5sum >(inputs_list_file))',
],
},
# Generate API 14 resources.
« no previous file with comments | « build/android/gyp/process_resources.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698