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

Unified Diff: build/host_jar.gypi

Issue 1708383002: Put generated non-test Java runner scripts into bin/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add used_by_test_only variable. Created 4 years, 10 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
Index: build/host_jar.gypi
diff --git a/build/host_jar.gypi b/build/host_jar.gypi
index 9f39d7c6b59a35d729239214294d76da64def92c..60777c6bd653d99464fa9d69ee36493c78e853e9 100644
--- a/build/host_jar.gypi
+++ b/build/host_jar.gypi
@@ -60,6 +60,7 @@
],
'enable_errorprone%': 0,
'errorprone_exe_path': '<(PRODUCT_DIR)/bin.java/chromium_errorprone',
+ 'used_only_by_tests%': 0,
},
'all_dependent_settings': {
'variables': {
@@ -122,7 +123,12 @@
'action_name': 'create_java_binary_script_<(_target_name)',
'message': 'Creating java binary script <(_target_name)',
'variables': {
- 'output': '<(PRODUCT_DIR)/bin/helper/<(_target_name)',
+ 'output': '<(PRODUCT_DIR)/bin/<(_target_name)',
+ 'conditions': [
+ ['used_only_by_tests == 1', {
+ 'output': '<(PRODUCT_DIR)/bin/helper/<(_target_name)',
+ }],
+ ],
},
'inputs': [
'<(DEPTH)/build/android/gyp/create_java_binary_script.py',

Powered by Google App Engine
This is Rietveld 408576698