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

Side by Side Diff: build/get_landmines.py

Issue 469623002: Revert of Make landmines work on local builds too (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 | « DEPS ('k') | build/gyp_chromium » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2013 The Chromium Authors. All rights reserved. 2 # Copyright 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """ 6 """
7 This file emits the list of reasons why a particular build needs to be clobbered 7 This file emits the list of reasons why a particular build needs to be clobbered
8 (or a list of 'landmines'). 8 (or a list of 'landmines').
9 """ 9 """
10 10
11 import optparse
11 import sys 12 import sys
12 13
13 import landmine_utils 14 import landmine_utils
14 15
15 16
16 builder = landmine_utils.builder 17 builder = landmine_utils.builder
17 distributor = landmine_utils.distributor 18 distributor = landmine_utils.distributor
18 gyp_defines = landmine_utils.gyp_defines 19 gyp_defines = landmine_utils.gyp_defines
19 gyp_msvs_version = landmine_utils.gyp_msvs_version 20 gyp_msvs_version = landmine_utils.gyp_msvs_version
20 platform = landmine_utils.platform 21 platform = landmine_utils.platform
21 22
22 23
23 def print_landmines(): 24 def print_landmines(target):
24 """ 25 """
25 ALL LANDMINES ARE EMITTED FROM HERE. 26 ALL LANDMINES ARE EMITTED FROM HERE.
27 target can be one of {'Release', 'Debug', 'Debug_x64', 'Release_x64'}.
26 """ 28 """
27 if (distributor() == 'goma' and platform() == 'win32' and 29 if (distributor() == 'goma' and platform() == 'win32' and
28 builder() == 'ninja'): 30 builder() == 'ninja'):
29 print 'Need to clobber winja goma due to backend cwd cache fix.' 31 print 'Need to clobber winja goma due to backend cwd cache fix.'
30 if platform() == 'android': 32 if platform() == 'android':
31 print 'Clobber: To avoid unresolved link errors on Breakpad roll.' 33 print 'Clobber: To avoid unresolved link errors on Breakpad roll.'
32 print 'Clobber: To get rid of generated files in the wrong package.' 34 print 'Clobber: To get rid of generated files in the wrong package.'
33 if platform() == 'win' and builder() == 'ninja': 35 if platform() == 'win' and builder() == 'ninja':
34 print 'Compile on cc_unittests fails due to symbols removed in r185063.' 36 print 'Compile on cc_unittests fails due to symbols removed in r185063.'
35 if platform() == 'linux' and builder() == 'ninja': 37 if platform() == 'linux' and builder() == 'ninja':
(...skipping 16 matching lines...) Expand all
52 print 'Need to clobber everything due to gen file moves in r175513 (Blink)' 54 print 'Need to clobber everything due to gen file moves in r175513 (Blink)'
53 if (platform() != 'ios'): 55 if (platform() != 'ios'):
54 print 'Clobber to get rid of obselete test plugin after r248358' 56 print 'Clobber to get rid of obselete test plugin after r248358'
55 print 'Clobber to rebuild GN files for V8' 57 print 'Clobber to rebuild GN files for V8'
56 print 'Need to clobber everything due to build_nexe change in nacl r13424' 58 print 'Need to clobber everything due to build_nexe change in nacl r13424'
57 print '[chromium-dev] PSA: clobber build needed for IDR_INSPECTOR_* compil...' 59 print '[chromium-dev] PSA: clobber build needed for IDR_INSPECTOR_* compil...'
58 print 'blink_resources.grd changed: crbug.com/400860' 60 print 'blink_resources.grd changed: crbug.com/400860'
59 61
60 62
61 def main(): 63 def main():
62 print_landmines() 64 parser = optparse.OptionParser()
65 parser.add_option('-t', '--target',
66 help=='Target for which the landmines have to be emitted')
67
68 options, args = parser.parse_args()
69
70 if args:
71 parser.error('Unknown arguments %s' % args)
72
73 print_landmines(options.target)
63 return 0 74 return 0
64 75
65 76
66 if __name__ == '__main__': 77 if __name__ == '__main__':
67 sys.exit(main()) 78 sys.exit(main())
OLDNEW
« no previous file with comments | « DEPS ('k') | build/gyp_chromium » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698