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

Unified Diff: build/common.gypi

Issue 9513003: Add target base_unittests_run and tools/isolate/isolate.py. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove directory from 'inputs' Created 8 years, 9 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.gyp ('k') | tools/isolate/PRESUBMIT.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 82b2bc0363ea0bca2d7779c7d832932091ed9749..7d56c79677929cf347fe6021c7b20f7bcfc48088 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -287,6 +287,14 @@
# with one of those tools.
'build_for_tool%': '',
+ # Whether tests targets should be run, archived or just have the
+ # dependencies verified. All the tests targets have the '_run' suffix,
+ # e.g. base_unittests_run runs the target base_unittests. The test target
+ # always calls tools/isolate/isolate.py. See the script's --help for more
+ # information and the valid --mode values. Meant to be overriden with
+ # GYP_DEFINES.
+ 'tests_run%': 'check',
+
'conditions': [
# TODO(epoger): Figure out how to set use_skia=1 for Mac outside of
# the 'conditions' clause. Initial attempts resulted in chromium and
@@ -476,6 +484,8 @@
'linux_use_gold_binary%': '<(linux_use_gold_binary)',
'linux_use_gold_flags%': '<(linux_use_gold_flags)',
'use_canvas_skia_skia%': '<(use_canvas_skia_skia)',
+ 'tests_run%': '<(tests_run)',
+
# Whether to build for Wayland display server
'use_wayland%': 0,
« no previous file with comments | « base/base.gyp ('k') | tools/isolate/PRESUBMIT.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698