Cleanup buildbot_annotated_steps.py to run and test at the top level.

client/tools/buildbot_annotated_steps.py

 # 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.
 
@@ skipping 16 matching lines @@
 
 # latest dartium location
 DARTIUM_VERSION_FILE = 'tests/drt/LAST_VERSION'
 DARTIUM_V_MATCHER = (
     'gs://dartium-archive/[^/]*/dartium-\w*-inc-([0-9]*).([0-9]*).zip')
 
 # Patterns are of the form "dart_client-linux-ia32-debug"
 BUILDER_PATTERN = r'dart_client-(\w+)-(\w+)-(\w+)'
 
 
-def GetBuildInfo(srcpath):
+def GetBuildInfo():
   """Returns a tuple (name, version, arch, mode, platform) where:
     - name: A name for the build - the buildbot host if a buildbot.
     - version: A version string corresponding to this build.
     - component: 'dartium' (default) or 'chromium'
     - mode: 'debug' or 'release' (default)
     - platform: 'linux' or 'mac'
   """
   name = None
   version = None
   mode = 'release'
   component = 'dartium'
   platform = 'linux'
 
   # Populate via builder environment variables.
   name = os.environ.get(BUILDER_NAME)
   version = os.environ.get(REVISION)
 
   if name:
     pattern = re.match(BUILDER_PATTERN, name)
     if pattern:
       platform = pattern.group(1)
       component = pattern.group(2)
       mode = pattern.group(3)
 
   # Fall back if not on builder.
   if not name:
     name = socket.gethostname().split('.')[0]
   if not version:
-    os.chdir(srcpath)
     pipe = subprocess.Popen(
         ['svnversion', '-n'], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
     output = pipe.communicate()
     if pipe.returncode == 0:
       version = output[0]
     else:
       version = 'unknown'
   return (name, version, component, mode, platform)
 
 
-def RunDartcCompiler(client_path, mode, outdir):
+def RunDartcCompiler(mode, outdir):
   """Compiles the client code to javascript for dartc tests."""
-  # Move to the client directory and call the build script
-  os.chdir(client_path)
   return subprocess.call(
-      [sys.executable, '../tools/build.py', '--mode=' + mode])
+      [sys.executable, './tools/build.py', '--mode=' + mode])
 
-def RunBrowserTests(client_path, component, mode, platform):
+def RunBrowserTests(component, mode, platform):
   """Runs the Dart client tests."""
   if platform == 'linux':
     cmd = ['xvfb-run']
   else:
     cmd = []
-  # Move to the client directory and call the test script
-  os.chdir(client_path)
-  cmd += [sys.executable, '../tools/test.py',
+  cmd += [sys.executable, './tools/test.py',
      '--component=' + component, '--mode=' + mode,
      '--time', '--report', '--progress=buildbot', '-v']
   return subprocess.call(cmd)
 
-def GetUtils(srcpath):
+def GetUtils():
   '''
   dynamically get the utils module
   We use a dynamic import for tools/util.py because we derive its location
   dynamically using sys.argv[0]. This allows us to run this script from
   different directories.
 
   args:
-  srcpath - the location of the source code to build
   '''
-  sys.path.append(os.path.abspath(os.path.join(srcpath, '..', 'tools')))
+  sys.path.append(os.path.abspath(os.path.join('.', 'tools')))
   utils = __import__('utils')
   return utils
 
-def GetOutDir(utils, mode, name):
+def GetOutDir(utils, mode):
   '''
   get the location to place the output
 
   args:
   utils - the tools/utils.py module
   mode - the mode release or debug
-  name - the name of the builder
   '''
-  return utils.GetBuildRoot(utils.GuessOS(), mode, name)
+  return utils.GetBuildRoot(utils.GuessOS(), mode, utils.ARCH_GUESS)
 
-def ProcessDartClientTests(srcpath, component, mode, platform, name):
+def ProcessDartClientTests(component, mode, platform, name):
   '''
   build and test the dart client applications
 
   args:
-  srcpath - the location of the source code to build
   component - the component we are testing against
   mode - the mode release or debug
   platform - the platform we are building for
   '''
   print 'ProcessDartClientTests'
   if component == 'chromium':
     print ('@@@BUILD_STEP dartc dart clients: %s@@@' % name)
 
-    utils = GetUtils(srcpath)
-    outdir = GetOutDir(utils, mode, "dartc")
-    status = RunDartcCompiler(srcpath, mode, outdir)
+    utils = GetUtils()
+    outdir = GetOutDir(utils, mode)
+    status = RunDartcCompiler(mode, outdir)
     if status != 0:
       return status
 
   if component == 'dartium':
-    version_file = os.path.join(srcpath, DARTIUM_VERSION_FILE)
-    if os.path.exists(version_file):
+    if os.path.exists(DARTIUM_VERSION_FILE):
       latest = open(version_file, 'r').read()
       match = re.match(DARTIUM_V_MATCHER, latest)
       if match:
         print '@@@BUILD_STEP vm r%s (dartium r%s)@@@' % (
             match.group(2), match.group(1))
   print '@@@BUILD_STEP browser unit tests@@@'
-  return RunBrowserTests(srcpath, component, mode, platform)
+  return RunBrowserTests(component, mode, platform)
 
-def ProcessTools(srcpath, mode, name, version):
+def ProcessTools(mode, name, version):
   '''
   build and test the tools
 
   args:
   srcpath - the location of the source code to build
   mode - the mode release or debug
   version - the svn version of the currently checked out code
   '''
   print 'ProcessTools'
 
-  toolsBuildScript = os.path.join(srcpath, '..', 'editor', 'build', 'build.py')
+  toolsBuildScript = os.path.join('.', 'editor', 'build', 'build.py')
 
   #TODO: debug statements to be removed in the future.
-  print "srcpath = " + srcpath
   print "mode = " + mode
   print "name = " + name
   print "version = " + version
   print "toolsBuildScript = " + os.path.abspath(toolsBuildScript)
 
-  utils = GetUtils(srcpath)
-  outdir = GetOutDir(utils, mode, "tools")
+  utils = GetUtils()
+  outdir = GetOutDir(utils, mode)
   cmds = [sys.executable, toolsBuildScript,
           '--mode=' + mode, '--revision=' + version,
           '--name=' + name, '--out=' + outdir]
   return subprocess.call(cmds)
 
-def ProcessFrog(srcpath):
+def ProcessFrog():
   '''
   build and test experimental frog build
-
-  args:
-  srcpath - the location of the source code to build
   '''
   print 'ProcessFrog'
 
   return subprocess.call([sys.executable,
-      os.path.join(srcpath, '..', 'frog',
-        'scripts', 'buildbot_annotated_steps.py')])
+      os.path.join('frog', 'scripts', 'buildbot_annotated_steps.py')])
 
 def main():
   print 'main'
   if len(sys.argv) == 0:
     print 'Script pathname not known, giving up.'
     return 1
 
   scriptdir = os.path.dirname(sys.argv[0])
-  srcpath = os.path.abspath(os.path.join(scriptdir, '..'))
+  # Get at the top-level directory. This script is in client/tools
+  os.chdir(os.path.abspath(os.path.join(scriptdir, os.pardir, os.pardir)))
 
-  (name, version, component, mode, platform) = GetBuildInfo(srcpath)
+  (name, version, component, mode, platform) = GetBuildInfo()
   if name == 'dart-editor':
-    status = ProcessTools(srcpath, mode, name, version)
-  #TODO(sigmund): remove this indirection once we update out bots
+    status = ProcessTools(mode, name, version)
+  #TODO(sigmund): remove this indirection once we update our bots
   elif name.startswith('frog'):
-    status = ProcessFrog(srcpath)
+    status = ProcessFrog()
   else:
-    status = ProcessDartClientTests(srcpath, component, mode, platform, name)
+    status = ProcessDartClientTests(component, mode, platform, name)
 
   if status:
     print '@@@STEP_FAILURE@@@'
 
   return status
 
 
 if __name__ == '__main__':
   sys.exit(main())