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

Side by Side Diff: site_scons/site_tools/chromium_builders.py

Issue 16614: Exclude .idl files from the generated compilable files lists. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 11 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """ 5 """
6 Tool module for adding, to a construction environment, Chromium-specific 6 Tool module for adding, to a construction environment, Chromium-specific
7 wrappers around Hammer builders. This gives us a central place for any 7 wrappers around Hammer builders. This gives us a central place for any
8 customization we need to make to the different things we build. 8 customization we need to make to the different things we build.
9 """ 9 """
10 10
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 top[i] = new 51 top[i] = new
52 52
53 import __builtin__ 53 import __builtin__
54 __builtin__.ChromeFileList = ChromeFileList 54 __builtin__.ChromeFileList = ChromeFileList
55 55
56 non_compilable_suffixes = { 56 non_compilable_suffixes = {
57 'LINUX' : set([ 57 'LINUX' : set([
58 '.h', 58 '.h',
59 '.dat', 59 '.dat',
60 '.rc', 60 '.rc',
61 '.idl',
61 ]), 62 ]),
62 'WINDOWS' : set([ 63 'WINDOWS' : set([
63 '.h', 64 '.h',
64 '.dat', 65 '.dat',
66 '.idl',
65 ]), 67 ]),
66 } 68 }
67 69
68 def compilable(env, file): 70 def compilable(env, file):
69 base, ext = os.path.splitext(str(file)) 71 base, ext = os.path.splitext(str(file))
70 if ext in non_compilable_suffixes[env['TARGET_PLATFORM']]: 72 if ext in non_compilable_suffixes[env['TARGET_PLATFORM']]:
71 return False 73 return False
72 return True 74 return True
73 75
74 def compilable_files(env, sources): 76 def compilable_files(env, sources):
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
156 env.AddMethod(ChromeLibrary) 158 env.AddMethod(ChromeLibrary)
157 env.AddMethod(ChromeStaticLibrary) 159 env.AddMethod(ChromeStaticLibrary)
158 env.AddMethod(ChromeSharedLibrary) 160 env.AddMethod(ChromeSharedLibrary)
159 env.AddMethod(ChromeObject) 161 env.AddMethod(ChromeObject)
160 env.AddMethod(ChromeMSVSFolder) 162 env.AddMethod(ChromeMSVSFolder)
161 env.AddMethod(ChromeMSVSProject) 163 env.AddMethod(ChromeMSVSProject)
162 env.AddMethod(ChromeMSVSSolution) 164 env.AddMethod(ChromeMSVSSolution)
163 165
164 def exists(env): 166 def exists(env):
165 return True 167 return True
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698