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

Side by Side Diff: scripts/slave/recipe_modules/chromium/api.py

Issue 2045403003: Try to fix the "Clobber" button in the buildbot UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build@master
Patch Set: Created 4 years, 6 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
« 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 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 import re 5 import re
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 from recipe_engine import util as recipe_util 8 from recipe_engine import util as recipe_util
9 9
10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder): 10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder):
(...skipping 472 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 483
484 def ensure_goma(self, canary=False): 484 def ensure_goma(self, canary=False):
485 goma_dir = self.m.goma.ensure_goma(canary=canary) 485 goma_dir = self.m.goma.ensure_goma(canary=canary)
486 if goma_dir: 486 if goma_dir:
487 # TODO(phajdan.jr): goma_dir should always be non-empty. 487 # TODO(phajdan.jr): goma_dir should always be non-empty.
488 self.c.gyp_env.GYP_DEFINES['gomadir'] = goma_dir 488 self.c.gyp_env.GYP_DEFINES['gomadir'] = goma_dir
489 self.c.compile_py.goma_dir = goma_dir 489 self.c.compile_py.goma_dir = goma_dir
490 490
491 def clobber_if_needed(self): 491 def clobber_if_needed(self):
492 """Add an explicit clobber step if requested.""" 492 """Add an explicit clobber step if requested."""
493 if self.c.clobber_before_runhooks: 493 # clobber_before_runhooks is true for bots that apply the 'clobber' config,
494 # that is for bots that do clobber bots on every build.
495 # properties.get('clobber') is true on bots that normally don't clobber,
496 # when the "Clobber" button in the buildbot UI is pressed.
497 if (self.c.clobber_before_runhooks or
498 self.m.properties.get('clobber') is not None):
494 self.m.file.rmtree('clobber', self.output_dir) 499 self.m.file.rmtree('clobber', self.output_dir)
495 500
496 def runhooks(self, **kwargs): 501 def runhooks(self, **kwargs):
497 """Run the build-configuration hooks for chromium.""" 502 """Run the build-configuration hooks for chromium."""
498 503
499 # runhooks might write things into the output directory, so clobber before 504 # runhooks might write things into the output directory, so clobber before
500 # that. 505 # that.
501 self.clobber_if_needed() 506 self.clobber_if_needed()
502 507
503 env = self.get_env() 508 env = self.get_env()
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
765 })) 770 }))
766 771
767 def get_annotate_by_test_name(self, test_name): 772 def get_annotate_by_test_name(self, test_name):
768 return 'graphing' 773 return 'graphing'
769 774
770 def download_lto_plugin(self): 775 def download_lto_plugin(self):
771 return self.m.python( 776 return self.m.python(
772 name='download LTO plugin', 777 name='download LTO plugin',
773 script=self.m.path['checkout'].join( 778 script=self.m.path['checkout'].join(
774 'build', 'download_gold_plugin.py')) 779 'build', 'download_gold_plugin.py'))
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