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

Unified Diff: client/tools/buildbot_annotated_steps.py

Issue 12450026: Added tools/bots/editor.py to build the editor (using gyp/build.py) and run editor tests (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 | tools/bots/bot.py » ('j') | 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 4ed1adab1c164cf616aa41449330ba461ac1c6f8..a3fa9861d3d537e5e375d873871d5d2342410927 100755
--- a/client/tools/buildbot_annotated_steps.py
+++ b/client/tools/buildbot_annotated_steps.py
@@ -1,9 +1,8 @@
+#!/usr/bin/python
# Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
# for details. All rights reserved. Use of this source code is governed by a
# BSD-style license that can be found in the LICENSE file.
-#!/usr/bin/python
-
# Copyright (c) 2011 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
@@ -100,12 +99,6 @@ def ProcessTools(mode, name, version):
# get the latest changed revision from the current repository sub-tree
version = GetLatestChangedRevision()
- #TODO: debug statements to be removed in the future.
- print "mode = " + mode
- print "name = " + name
- print "version = " + version
- print "toolsBuildScript = " + os.path.abspath(toolsBuildScript)
-
utils = GetUtils()
outdir = GetOutDir(utils, mode)
cmds = [sys.executable, toolsBuildScript,
@@ -184,8 +177,6 @@ def GetLatestChangedRevision():
return svnRev
def main():
- print 'main'
-
if len(sys.argv) == 0:
print 'Script pathname not known, giving up.'
return 1
@@ -208,7 +199,18 @@ def main():
# root directory, set JAVA_HOME based on that.
FixJavaHome()
if name.startswith('dart-editor'):
- status = ProcessTools('release', name, version)
+ # 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) or status
+ # In case we're an FYI builder, run 'tools/bots/editor.py' as well
+ if is_fyi:
+ status = ProcessBot(name, 'editor')
elif name.startswith('pub-'):
status = ProcessBot(name, 'pub')
elif name.startswith('vm-android'):
« no previous file with comments | « no previous file | tools/bots/bot.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698