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

Unified Diff: tools/testing/perf_testing/create_graph.py

Issue 9325002: Fix windows performance tests. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: '' Created 8 years, 11 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 | « client/tests/client/samples/smoketest/dartWebBase_tests.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/perf_testing/create_graph.py
===================================================================
--- tools/testing/perf_testing/create_graph.py (revision 3854)
+++ tools/testing/perf_testing/create_graph.py (working copy)
@@ -92,6 +92,10 @@
run_cmd(['svn', 'revert', os.path.join(os.getcwd(), 'frog', 'minfrog')])
run_cmd(['gclient', 'sync'])
+
+ def on_rm_error(func, path, exc_info):
Emily Fortuna 2012/02/02 18:24:09 On windows, the output directory is marked as "Rea
Siggi Cherem (dart-lang) 2012/02/02 18:29:58 It would be great to put this exact comment on the
Emily Fortuna 2012/02/02 18:51:00 Done.
+ 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
@@ -100,7 +104,7 @@
# 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')))
+ utils.GetBuildRoot(utils.GuessOS(), 'release', 'ia32')), onerror=on_rm_error)
lines = run_cmd([os.path.join('.', 'tools', 'build.py'), '-m', 'release',
'--arch=ia32', 'create_sdk'])
« no previous file with comments | « client/tests/client/samples/smoketest/dartWebBase_tests.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698