Fix path previously only intended for Windows. (Now runs also if the out directory doesn't exist).

tools/testing/perf_testing/run_perf_tests.py

 #!/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.
 
 import datetime
 import getpass
 import math
 from matplotlib.font_manager import FontProperties
 import matplotlib.pyplot as plt
 import optparse
 import os
 from os.path import dirname, abspath
 import platform
 import shutil
+import stat
 import subprocess
 import time
 import traceback
 import sys

[vsm, 2012/03/29 18:37:03]

Nit: sys before time for alpha order.

 
 TOOLS_PATH = os.path.join(dirname(dirname(dirname(abspath(__file__)))))
 sys.path.append(TOOLS_PATH)
 import utils
 
 """This script runs to track performance and correctness progress of 
 different svn revisions. It tests to see if there a newer version of the code on
 the server, and will sync and run the performance tests if so."""
 
 DART_INSTALL_LOCATION = os.path.join(dirname(abspath(__file__)),
@@ skipping 65 matching lines @@
   os.chdir(DART_INSTALL_LOCATION)
   #Revert our newly built minfrog to prevent conflicts when we update
   run_cmd(['svn', 'revert',  os.path.join(os.getcwd(), 'frog', 'minfrog')])
 
   run_cmd(['gclient', 'sync'])
  
   # On Windows, the output directory is marked as "Read Only," which causes an 
   # error to be thrown when we use shutil.rmtree. This helper function changes
   # the permissions so we can still delete the directory.
   def on_rm_error(func, path, exc_info):
-    os.chmod(path, stat.S_IWRITE)
-    os.unlink(path)
+    if os.path.exists(path):
+      os.chmod(path, stat.S_IWRITE)
+      os.unlink(path)
   # TODO(efortuna): building the sdk locally is a band-aid until all build
   # platform SDKs are hosted in Google storage. Pull from https://sandbox.
   # google.com/storage/?arg=dart-dump-render-tree#dart-dump-render-tree%2Fsdk
   # eventually.
   # TODO(efortuna): Currently always building ia32 architecture because we don't
   # have test statistics for what's passing on x64. Eliminate arch specification
   # when we have tests running on x64, too.
   shutil.rmtree(os.path.join(os.getcwd(), 
       utils.GetBuildRoot(utils.GuessOS(), 'release', 'ia32')), 
       onerror=on_rm_error)
@@ skipping 647 matching lines @@
     while True:
       if has_new_code():
         run_test_sequence(cl, size, language, perf, username, password)
       else:
         time.sleep(SLEEP_TIME)
   else:
     run_test_sequence(cl, size, language, perf, username, password)
 
 if __name__ == '__main__':
   main()