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

Unified Diff: client/tools/buildbot_annotated_steps.py

Issue 13461007: Don't run the old annotated step script on our dart-editor-fyi-* builders (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/tools/buildbot_annotated_steps.py
diff --git a/client/tools/buildbot_annotated_steps.py b/client/tools/buildbot_annotated_steps.py
index c4242550291c7a4b4da8aaebc1c28fb5adcb180b..bbb64471675c3cbc0ccfa7cbea570812ae6ede50 100755
--- a/client/tools/buildbot_annotated_steps.py
+++ b/client/tools/buildbot_annotated_steps.py
@@ -203,16 +203,14 @@ def main():
if name.startswith('dart-editor'):
# TODO(kustermann,ricow): This is a temporary hack until we can safely
# enable it on main waterfall. We need to remove this eventually
- is_fyi = False
if name.startswith('dart-editor-fyi'):
match = re.search('dart-editor-fyi(.*)', name)
name = 'dart-editor' + match.group(1)
- is_fyi = True
- # Run the old annotated steps script first.
- status = ProcessTools('release', name, version)
- # In case we're an FYI builder, run 'tools/bots/editor.py' as well
- if is_fyi:
- status = ProcessBot(name, 'editor') or status
+ # In case we're an FYI builder, run 'tools/bots/editor.py'.
+ status = ProcessBot(name, 'editor')
+ else:
+ # Run the old annotated steps script
+ status = ProcessTools('release', name, version)
elif name.startswith('pub-'):
status = ProcessBot(name, 'pub')
elif name.startswith('vm-android'):
« 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