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

Unified Diff: build/nocompile.gypi

Issue 678263003: Switch to clang for nocompile tests and rebaseline existing results. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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: build/nocompile.gypi
diff --git a/build/nocompile.gypi b/build/nocompile.gypi
index f9021ae3797c51d667d61a7532cb5410221c8f4d..8c0f288b5a55f70fcfb38013e509b0b0b4b549c4 100644
--- a/build/nocompile.gypi
+++ b/build/nocompile.gypi
@@ -60,7 +60,7 @@
# TODO(awong): Disabled until http://crbug.com/105388 is resolved.
'sources/': [['exclude', '\\.nc$']],
'conditions': [
- [ 'OS=="linux" and clang==0', {
+ [ 'OS!="win" and clang==1', {
'rules': [
{
'variables': {
@@ -90,7 +90,7 @@
],
}, {
'sources/': [['exclude', '\\.nc$']]
danakj 2014/11/03 16:05:13 btw isn't this redundant with L61?
dcheng 2014/11/03 18:58:18 Think of line 61 as a quick way to ifdef out this
danakj 2014/11/03 19:00:14 Oh I see, that's why the TODO is there.
- }], # 'OS=="linux" and clang=="0"'
+ }], # 'OS!="win" and clang=="1"'
],
}

Powered by Google App Engine
This is Rietveld 408576698