Closed
Bug 973937
Opened 11 years ago
Closed 11 years ago
[mozcrash] check_for_crashes() doesn't save .extra file to dump_save_path
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla30
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Whiteboard: [mozmill-2.0.6+])
Attachments
(1 file)
7.52 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
At the moment only the .dmp file is getting saved to the specified dump_save_path. But to let the crash reporter send the crash information also the corresponding .extra file is needed.
Assignee | ||
Comment 1•11 years ago
|
||
I will take this given that we need this fixed for Mozmill 2.0.6.
Assignee | ||
Comment 2•11 years ago
|
||
That updates mozcrash to also safe off the .extra file if existent. The patch also adds tests. I will push to try in a bit, but feel free to review already.
Attachment #8377683 -
Flags: review?(ted)
Assignee | ||
Comment 3•11 years ago
|
||
Updated•11 years ago
|
Attachment #8377683 -
Flags: review?(ted) → review+
Assignee | ||
Comment 4•11 years ago
|
||
I cannot land those bits given that we need the merge of mozbase done via bug 949600.
Depends on: 949600
Assignee | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/5dea7efc7515
Please leave the bug open because I have to release a new version of mozcrash.
Comment 6•11 years ago
|
||
Assignee | ||
Comment 7•11 years ago
|
||
mozcrash 0.12 has been released to pypi:
running upload
Submitting dist/mozcrash-0.12.tar.gz to http://pypi.python.org/pypi
Server response (200): OK
You need to log in
before you can comment on or make changes to this bug.
Description
•