| OLD | NEW |
| 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 Loading... |
| 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 } |
| OLD | NEW |