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

Unified Diff: tests/stubout_mode/nacl.scons

Issue 625923004: Delete old x86 validator. (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: Created 6 years, 2 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
Index: tests/stubout_mode/nacl.scons
diff --git a/tests/stubout_mode/nacl.scons b/tests/stubout_mode/nacl.scons
index d44ee22b9080603b33a7b72d86d170ad589bb811..8cc0d9138c580a78f3752fb48bea331f944f4c40 100644
--- a/tests/stubout_mode/nacl.scons
+++ b/tests/stubout_mode/nacl.scons
@@ -45,7 +45,9 @@ partly_invalid_nexe = env.ComponentProgram('partly_invalid', 'partly_invalid.c',
# R-DFA does not support stubout mode but we need to create at least file
# partly_invalid.nexe used in PPAPI tests thus we first create a rule for
# this file and only then bail out.
-if env.Bit('validator_ragel'):
+# TODO(shyamsundarr): given that neither arm not x86 support stubout mode
+# and mips tests aren't run on bots, investigate deleting.
+if env.Bit('build_x86'):
Mark Seaborn 2014/10/06 17:47:55 This disables the rest of the file on both x86 and
shyamsundarr 2014/10/06 23:06:57 Done here for now but i will break it out if i spl
Return()
partly_invalid_stubout_nexe = env.File('partly_invalid_stubout${PROGSUFFIX}')

Powered by Google App Engine
This is Rietveld 408576698