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

Unified Diff: pylib/gyp/generator/ninja.py

Issue 10270020: ninja windows: make sure that .s files aren't passed to compiler (Closed) Base URL: https://gyp.googlecode.com/svn/trunk
Patch Set: add comment on .s filter 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 | « no previous file | test/win/asm-files/asm-files.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pylib/gyp/generator/ninja.py
diff --git a/pylib/gyp/generator/ninja.py b/pylib/gyp/generator/ninja.py
index 54c79e5c28b3669647a19bd5b01ef558379a4d45..07a5ac3ef4f72d8f5a272e46bfc48817d81730c6 100644
--- a/pylib/gyp/generator/ninja.py
+++ b/pylib/gyp/generator/ninja.py
@@ -716,7 +716,11 @@ class NinjaWriter:
obj_ext = self.obj_ext
if ext in ('cc', 'cpp', 'cxx'):
command = 'cxx'
- elif ext in ('c', 's', 'S'):
+ elif ext == 'c' or (ext in ('s', 'S') and self.flavor != 'win'):
+ # TODO(scottmg): .s files won't be handled by the Windows compiler,
Nico 2012/04/30 20:54:48 Nit: s/,/./ at the end of this line, s/but we/We/
scottmg 2012/04/30 20:56:50 Done.
+ # but we could add support for .asm, though that's only supported on
+ # x86. Currently not used in Chromium in favor of other third-party
+ # assemblers.
command = 'cc'
elif self.flavor == 'mac' and ext == 'm':
command = 'objc'
« no previous file with comments | « no previous file | test/win/asm-files/asm-files.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698