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

Side by Side Diff: build/isolate.gypi

Issue 111393006: Revert of Roll swarming_client @ 9cc42c3d4d. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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 | Annotate | Revision Log
« no previous file with comments | « build/android/pylib/gtest/setup.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 15 matching lines...) Expand all
26 # ], 26 # ],
27 # }], 27 # }],
28 # ], 28 # ],
29 # 29 #
30 # Note: foo_test.isolate is included and a source file. It is an inherent 30 # Note: foo_test.isolate is included and a source file. It is an inherent
31 # property of the .isolate format. This permits to define GYP variables but is 31 # property of the .isolate format. This permits to define GYP variables but is
32 # a stricter format than GYP so isolate.py can read it. 32 # a stricter format than GYP so isolate.py can read it.
33 # 33 #
34 # The generated .isolated file will be: 34 # The generated .isolated file will be:
35 # <(PRODUCT_DIR)/foo_test.isolated 35 # <(PRODUCT_DIR)/foo_test.isolated
36 #
37 # See http://dev.chromium.org/developers/testing/isolated-testing/for-swes
38 # for more information.
39 36
40 { 37 {
41 'rules': [ 38 'rules': [
42 { 39 {
43 'rule_name': 'isolate', 40 'rule_name': 'isolate',
44 'extension': 'isolate', 41 'extension': 'isolate',
45 'inputs': [ 42 'inputs': [
46 # Files that are known to be involved in this step. 43 # Files that are known to be involved in this step.
47 '<(DEPTH)/tools/swarming_client/isolate.py', 44 '<(DEPTH)/tools/swarming_client/isolate.py',
48 '<(DEPTH)/tools/swarming_client/run_isolated.py', 45 '<(DEPTH)/tools/swarming_client/run_isolated.py',
46 '<(DEPTH)/tools/swarming_client/googletest/run_test_cases.py',
49 47
50 # Disable file tracking by the build driver for now. This means the 48 # Disable file tracking by the build driver for now. This means the
51 # project must have the proper build-time dependency for their runtime 49 # project must have the proper build-time dependency for their runtime
52 # dependency. This improves the runtime of the build driver since it 50 # dependency. This improves the runtime of the build driver since it
53 # doesn't have to stat() all these files. 51 # doesn't have to stat() all these files.
54 # 52 #
55 # More importantly, it means that even if a isolate_dependency_tracked 53 # More importantly, it means that even if a isolate_dependency_tracked
56 # file is missing, for example if a file was deleted and the .isolate 54 # file is missing, for example if a file was deleted and the .isolate
57 # file was not updated, that won't break the build, especially in the 55 # file was not updated, that won't break the build, especially in the
58 # case where foo_tests_run is not built! This should be reenabled once 56 # case where foo_tests_run is not built! This should be reenabled once
59 # the switch-over to running tests on Swarm is completed. 57 # the switch-over to running tests on Swarm is completed.
60 #'<@(isolate_dependency_tracked)', 58 #'<@(isolate_dependency_tracked)',
61 ], 59 ],
62 'outputs': [ 60 'outputs': [
63 '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated', 61 '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated',
64 ], 62 ],
65 'action': [ 63 'action': [
66 'python', 64 'python',
67 '<(DEPTH)/tools/swarming_client/isolate.py', 65 '<(DEPTH)/tools/swarming_client/isolate.py',
68 '<(test_isolation_mode)', 66 '<(test_isolation_mode)',
69 # Variables should use the -V FOO=<(FOO) form so frequent values, 67 # Variables should use the -V FOO=<(FOO) form so frequent values,
70 # like '0' or '1', aren't stripped out by GYP. 68 # like '0' or '1', aren't stripped out by GYP.
71 '--path-variable', 'PRODUCT_DIR', '<(PRODUCT_DIR) ', 69 '--variable', 'PRODUCT_DIR', '<(PRODUCT_DIR) ',
72 '--config-variable', 'OS=<(OS)', 70 '--variable', 'OS=<(OS)',
73 '--result', '<@(_outputs)', 71 '--result', '<@(_outputs)',
74 '--isolate', '<(RULE_INPUT_PATH)', 72 '--isolate', '<(RULE_INPUT_PATH)',
75 ], 73 ],
76 'conditions': [ 74 'conditions': [
77 # Note: When gyp merges lists, it appends them to the old value. 75 # Note: When gyp merges lists, it appends them to the old value.
78 ['OS=="mac"', { 76 ['OS=="mac"', {
79 # <(mac_product_name) can contain a space, so don't use FOO=<(FOO) 77 # <(mac_product_name) can contain a space, so don't use FOO=<(FOO)
80 # form. 78 # form.
81 'action': [ 79 'action': [ '--variable', 'mac_product_name', '<(mac_product_name)' ],
82 '--extra-variable', 'mac_product_name', '<(mac_product_name)',
83 ],
84 }], 80 }],
85 ["test_isolation_outdir==''", { 81 ["test_isolation_outdir==''", {
86 # GYP will eliminate duplicate arguments so '<(PRODUCT_DIR)' cannot 82 # GYP will eliminate duplicate arguments so '<(PRODUCT_DIR)' cannot
87 # be provided twice. To work around this behavior, append '/'. 83 # be provided twice. To work around this behavior, append '/'.
88 # 84 #
89 # Also have a space after <(PRODUCT_DIR) or visual studio will 85 # Also have a space after <(PRODUCT_DIR) or visual studio will
90 # escape the argument wrappping " with the \ and merge it into 86 # escape the argument wrappping " with the \ and merge it into
91 # the following arguments. 87 # the following arguments.
92 'action': [ '--outdir', '<(PRODUCT_DIR)/ ' ], 88 'action': [ '--outdir', '<(PRODUCT_DIR)/ ' ],
93 }, { 89 }, {
94 'action': [ '--outdir', '<(test_isolation_outdir)' ], 90 'action': [ '--outdir', '<(test_isolation_outdir)' ],
95 }], 91 }],
96 ['test_isolation_fail_on_missing == 0', { 92 ['test_isolation_fail_on_missing == 0', {
97 'action': ['--ignore_broken_items'], 93 'action': ['--ignore_broken_items'],
98 }, 94 },
99 ], 95 ],
100 ], 96 ],
101 97
102 'msvs_cygwin_shell': 0, 98 'msvs_cygwin_shell': 0,
103 }, 99 },
104 ], 100 ],
105 } 101 }
OLDNEW
« no previous file with comments | « build/android/pylib/gtest/setup.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698