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

Issue 5131005: Add python's version in breakpad stack traces (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
jam
CC:
chromium-reviews, M-A Ruel
Visibility:
Public.

Description

Add python's version in breakpad stack traces BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67314

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M breakpad.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
10 years, 1 month ago (2010-11-20 02:50:51 UTC) #1
jam
10 years, 1 month ago (2010-11-20 03:04:47 UTC) #2
lgtm

On Fri, Nov 19, 2010 at 6:50 PM, <maruel@chromium.org> wrote:

> Reviewers: John Abd-El-Malek,
>
> Description:
> Add python's version in breakpad stack traces
>
> BUG=
> TEST=
>
> Please review this at http://codereview.chromium.org/5131005/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
>
> Affected files:
>  M breakpad.py
>
>
> Index: breakpad.py
> diff --git a/breakpad.py b/breakpad.py
> index
>
f8f1a1b5aa77f95b43ef74359a157c0a9396ad4f..e6f4446b939faebc0a6d0dfdeabb94d78bd51b7b
> 100644
> --- a/breakpad.py
> +++ b/breakpad.py
> @@ -40,6 +40,7 @@ def SendStack(last_tb, stack, url=None):
>         'exception': last_tb,
>         'host': socket.getfqdn(),
>         'cwd': os.getcwd(),
> +        'version': sys.version,
>     }
>     # No exception type(s) specified
>     # pylint: disable=W0702
>
>
>

Powered by Google App Engine
This is Rietveld 408576698