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

Issue 2815553003: Add support for core dump archiving on macos (Closed)

Created:
3 years, 8 months ago by kustermann
Modified:
3 years, 8 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : wrap in try-catch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -19 lines) Patch
M tools/utils.py View 1 8 chunks +38 lines, -19 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
kustermann
First attempt and making it work!
3 years, 8 months ago (2017-04-11 08:05:44 UTC) #2
kustermann
3 years, 8 months ago (2017-04-12 08:58:54 UTC) #4
Vyacheslav Egorov (Google)
lgtm https://codereview.chromium.org/2815553003/diff/1/tools/utils.py File tools/utils.py (right): https://codereview.chromium.org/2815553003/diff/1/tools/utils.py#newcode961 tools/utils.py:961: os.unlink(core) maybe wrap this in try: for MacOS ...
3 years, 8 months ago (2017-04-18 11:12:51 UTC) #5
kustermann
https://codereview.chromium.org/2815553003/diff/1/tools/utils.py File tools/utils.py (right): https://codereview.chromium.org/2815553003/diff/1/tools/utils.py#newcode961 tools/utils.py:961: os.unlink(core) On 2017/04/18 11:12:51, Vyacheslav Egorov (Google) wrote: > ...
3 years, 8 months ago (2017-04-19 08:36:13 UTC) #6
kustermann
3 years, 8 months ago (2017-04-19 08:36:31 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
a7e6da3a93f93da03f8fee94b1c0175efe7a641d (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698