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

Issue 20097: Allow purify scripts to take a --report_dir argument which specifies where th... (Closed)

Created:
11 years, 10 months ago by Erik does not do reviews
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Allow purify scripts to take a --report_dir argument which specifies where the output data is written to. Also, add a --buildbot flag which will default the report_dir to a specific directory on chrome-web. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=9286

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Total comments: 1

Patch Set 6 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -28 lines) Patch
M tools/purify/chrome_tests.py View 1 2 3 4 5 4 chunks +31 lines, -2 lines 0 comments Download
M tools/purify/purify_analyze.py View 1 2 3 11 chunks +36 lines, -21 lines 0 comments Download
M tools/purify/purify_test.py View 1 2 3 5 chunks +9 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Erik does not do reviews
11 years, 10 months ago (2009-02-05 21:52:07 UTC) #1
Nicolas Sylvain
11 years, 10 months ago (2009-02-06 00:03:00 UTC) #2
LG with the comment

http://codereview.chromium.org/20097/diff/15/1014
File tools/purify/chrome_tests.py (right):

http://codereview.chromium.org/20097/diff/15/1014#newcode68
Line 68: chrome_web_dir = "//chrome-web.corp.google.com/chrome-bot/www/purify"
you need to do 
import chromium_config as config
chrome_web_dir = config.Archive.purify_test_result_archive
I already created it.

Powered by Google App Engine
This is Rietveld 408576698