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

Unified Diff: net/net.gyp

Issue 10019014: Convert isolate.py to exclusively use .isolate files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Do not read as binary Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/base_unittests.isolate ('k') | net/net_unittests.isolate » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/net.gyp
diff --git a/net/net.gyp b/net/net.gyp
index f4ab7b571e1defbb1a89ae77c840aadfc9e9fbaf..d33fc2290eeef9dfd3add2fb041be7ed35ec1d5d 100644
--- a/net/net.gyp
+++ b/net/net.gyp
@@ -1356,63 +1356,28 @@
'dependencies': [
'net_unittests',
],
+ 'includes': [
+ 'net_unittests.isolate',
+ ],
'actions': [
{
- 'action_name': 'response_file',
- 'inputs': [
- '<(PRODUCT_DIR)/net_unittests<(EXECUTABLE_SUFFIX)',
- '<(DEPTH)/testing/test_env.py',
- '<(DEPTH)/testing/xvfb.py',
- '<(DEPTH)/third_party/pyftpdlib/src/pyftpdlib/__init__.py',
- '<(DEPTH)/third_party/pyftpdlib/src/pyftpdlib/ftpserver.py',
- ],
- 'conditions': [
- ['OS=="linux"', {
- 'inputs': [
- '<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
- ],
- }],
- ],
- 'outputs': [
- '<(PRODUCT_DIR)/net_unittests.inputs',
- ],
- 'action': [
- 'python',
- '-c',
- 'import sys; '
- 'open(sys.argv[1], \'w\').write(\'\\n\'.join(sys.argv[2:]))',
- '<@(_outputs)',
- '<@(_inputs)',
- ],
- },
- {
'action_name': 'isolate',
'inputs': [
- '<(PRODUCT_DIR)/net_unittests.inputs',
+ 'net_unittests.isolate',
+ '<@(isolate_dependency_tracked)',
],
'outputs': [
'<(PRODUCT_DIR)/net_unittests.results',
],
'action': [
'python',
- '<(DEPTH)/tools/isolate/isolate.py',
- '--mode=<(tests_run)',
- '--root', '<(DEPTH)',
+ '../tools/isolate/isolate.py',
+ '--mode', '<(tests_run)',
+ '--variable', 'DEPTH=<(DEPTH)',
+ '--variable', 'PRODUCT_DIR=<(PRODUCT_DIR)',
+ '--variable', 'OS=<(OS)',
'--result', '<@(_outputs)',
- '--files', '<@(_inputs)',
- # Directories can't be tracked by build tools (make, msbuild, xcode,
- # etc) so we just put it on the command line without specifying it
- # as an input.
- # TODO(maruel): Revisit the support for this at all and list each
- # individual test files instead.
- 'data/',
- 'tools/testserver/',
- '<(DEPTH)/third_party/tlslite/tlslite/',
- '--',
- # Wraps net_unittests under xvfb.
- '<(DEPTH)/testing/xvfb.py',
- '<(PRODUCT_DIR)',
- '<(PRODUCT_DIR)/net_unittests<(EXECUTABLE_SUFFIX)',
+ 'net_unittests.isolate',
],
},
],
« no previous file with comments | « base/base_unittests.isolate ('k') | net/net_unittests.isolate » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698