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

Issue 1061663002: Adding batch file and scripts to get new /analyze warnings. (Closed)

Created:
5 years, 8 months ago by brucedawson
Modified:
5 years, 8 months ago
Reviewers:
cpu_(ooo_6.6-7.5)
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Adding batch file and scripts to get new /analyze warnings. This set of tools makes it easy and efficient to explore new /analyze warnings, looking for new warnings. If you set ANALYZE_REPO and then run retrieve_latest_warnings.bat then it will retrieve the latest /analyze build results, summarize them, compare them to the previous results (if any), and sync the requested repo (git checkout and gclient sync) to the specified changelist. The new warnings can then be copied to the VS output window (using fromclip.exe, available separately) for easy browsing. BUG=427616 Committed: https://crrev.com/273467edbc595491263cc799f810615e7fafde5a Cr-Commit-Position: refs/heads/master@{#324707}

Patch Set 1 #

Patch Set 2 : Tweaked warning regex to work with VS 2015, and added initial OWNERS and README files. #

Patch Set 3 : Added license file, comments, support for errors and LNK warnings. #

Patch Set 4 : Set executable bit on retrieve_latest_warnings.bat. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+518 lines, -0 lines) Patch
A tools/win/new_analyze_warnings/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
A tools/win/new_analyze_warnings/README View 1 2 1 chunk +20 lines, -0 lines 0 comments Download
A tools/win/new_analyze_warnings/retrieve_latest_warnings.bat View 1 2 3 1 chunk +71 lines, -0 lines 0 comments Download
A tools/win/new_analyze_warnings/retrieve_warnings.py View 1 2 1 chunk +100 lines, -0 lines 0 comments Download
A tools/win/new_analyze_warnings/warning_diff.py View 1 2 1 chunk +165 lines, -0 lines 0 comments Download
A tools/win/new_analyze_warnings/warnings_by_type.py View 1 2 1 chunk +160 lines, -0 lines 0 comments Download

Messages

Total messages: 13 (5 generated)
brucedawson
I decided to put my /analyze warning-retrieval code into a CR just so that it ...
5 years, 8 months ago (2015-04-03 20:36:49 UTC) #2
cpu_(ooo_6.6-7.5)
turns out I don't know how to read python, but I can OK this CL ...
5 years, 8 months ago (2015-04-09 00:01:16 UTC) #3
cpu_(ooo_6.6-7.5)
lgtm
5 years, 8 months ago (2015-04-10 00:59:29 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1061663002/20001
5 years, 8 months ago (2015-04-10 01:00:36 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/55352)
5 years, 8 months ago (2015-04-10 01:11:36 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1061663002/60001
5 years, 8 months ago (2015-04-10 22:08:40 UTC) #11
commit-bot: I haz the power
Committed patchset #4 (id:60001)
5 years, 8 months ago (2015-04-10 23:08:07 UTC) #12
commit-bot: I haz the power
5 years, 8 months ago (2015-04-10 23:10:51 UTC) #13
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/273467edbc595491263cc799f810615e7fafde5a
Cr-Commit-Position: refs/heads/master@{#324707}

Powered by Google App Engine
This is Rietveld 408576698