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

Side by Side Diff: build/isolate.gypi

Issue 1062363002: Add comments about --config-vars that need to be in sync. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: maruel Created 5 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 unified diff | Download patch
« no previous file with comments | « build/android/pylib/utils/isolator.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # This file is meant to be included into a target to provide a rule 5 # This file is meant to be included into a target to provide a rule
6 # to "build" .isolate files into a .isolated file. 6 # to "build" .isolate files into a .isolated file.
7 # 7 #
8 # To use this, create a gyp target with the following form: 8 # To use this, create a gyp target with the following form:
9 # 'conditions': [ 9 # 'conditions': [
10 # ['test_isolation_mode != "noop"', { 10 # ['test_isolation_mode != "noop"', {
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 'action': [ 54 'action': [
55 'python', 55 'python',
56 '<(DEPTH)/tools/isolate_driver.py', 56 '<(DEPTH)/tools/isolate_driver.py',
57 '<(test_isolation_mode)', 57 '<(test_isolation_mode)',
58 '--isolated', '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated', 58 '--isolated', '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated',
59 '--isolate', '<(RULE_INPUT_PATH)', 59 '--isolate', '<(RULE_INPUT_PATH)',
60 60
61 # Variables should use the -V FOO=<(FOO) form so frequent values, 61 # Variables should use the -V FOO=<(FOO) form so frequent values,
62 # like '0' or '1', aren't stripped out by GYP. Run 'isolate.py help' for 62 # like '0' or '1', aren't stripped out by GYP. Run 'isolate.py help' for
63 # more details. 63 # more details.
64 #
65 # This list needs to be kept in sync with the cmd line options
66 # in src/build/android/pylib/gtest/setup.py.
67 64
68 # Path variables are used to replace file paths when loading a .isolate 65 # Path variables are used to replace file paths when loading a .isolate
69 # file 66 # file
70 '--path-variable', 'DEPTH', '<(DEPTH)', 67 '--path-variable', 'DEPTH', '<(DEPTH)',
71 '--path-variable', 'PRODUCT_DIR', '<(PRODUCT_DIR) ', 68 '--path-variable', 'PRODUCT_DIR', '<(PRODUCT_DIR) ',
72 69
73 # Extra variables are replaced on the 'command' entry and on paths in 70 # Extra variables are replaced on the 'command' entry and on paths in
74 # the .isolate file but are not considered relative paths. 71 # the .isolate file but are not considered relative paths.
75 '--extra-variable', 'version_full=<(version_full)', 72 '--extra-variable', 'version_full=<(version_full)',
76 73
74 # Note: This list must match DefaultConfigVariables()
75 # in build/android/pylib/utils/isolator.py
77 '--config-variable', 'CONFIGURATION_NAME=<(CONFIGURATION_NAME)', 76 '--config-variable', 'CONFIGURATION_NAME=<(CONFIGURATION_NAME)',
78 '--config-variable', 'OS=<(OS)', 77 '--config-variable', 'OS=<(OS)',
79 '--config-variable', 'asan=<(asan)', 78 '--config-variable', 'asan=<(asan)',
80 '--config-variable', 'chromeos=<(chromeos)', 79 '--config-variable', 'chromeos=<(chromeos)',
81 '--config-variable', 'component=<(component)', 80 '--config-variable', 'component=<(component)',
82 '--config-variable', 'disable_nacl=<(disable_nacl)', 81 '--config-variable', 'disable_nacl=<(disable_nacl)',
83 '--config-variable', 'enable_plugins=<(enable_plugins)', 82 '--config-variable', 'enable_plugins=<(enable_plugins)',
84 '--config-variable', 'fastbuild=<(fastbuild)', 83 '--config-variable', 'fastbuild=<(fastbuild)',
85 '--config-variable', 'icu_use_data_file_flag=<(icu_use_data_file_flag)', 84 '--config-variable', 'icu_use_data_file_flag=<(icu_use_data_file_flag)',
86 # TODO(kbr): move this to chrome_tests.gypi:gles2_conform_tests_run 85 # TODO(kbr): move this to chrome_tests.gypi:gles2_conform_tests_run
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 ], 118 ],
120 }, { 119 }, {
121 'outputs': [ 120 'outputs': [
122 '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated', 121 '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated',
123 ], 122 ],
124 }], 123 }],
125 ], 124 ],
126 }, 125 },
127 ], 126 ],
128 } 127 }
OLDNEW
« no previous file with comments | « build/android/pylib/utils/isolator.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698