Closed
Bug 1450969
Opened 6 years ago
Closed 6 years ago
fix paths for MINIDUMP_STACKWALK for Windows build tasks
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: froydnj, Assigned: froydnj)
References
Details
Attachments
(1 file)
4.70 KB,
patch
|
RyanVM
:
review+
|
Details | Diff | Splinter Review |
If you happen to be unfortunate enough to be testing changes that crash xpcshell during the xpcshell self-tests, you'll be presented with error messages like: PROCESS-CRASH | test_child_assertions.js | application crashed [unknown top frame] Crash dump filename: c:\users\task_1522676338\appdata\local\temp\xpc-other-mtot6h\cfaea928-e995-4430-baf9-0066c6b91be9.dmp MINIDUMP_STACKWALK binary not found: z:\build\build\tools\breakpad\win64\minidump_stackwalk.exe and then be told that, essentially, "your test failed because it failed", which is unhelpful for figuring out what went wrong. Apparently we had MINIDUMP_STACKWALK set at one point, and then it got moved. Let's fix this situation by a) downloading minidump_stackwalk out of tooltool (our test harnesses do this already); and b) pointing MINIDUMP_STACKWALK at the correct location. With these changes, we can once again get crash stacks if xpcshell (and probably a few other things) fail their self tests.
Assignee | ||
Comment 1•6 years ago
|
||
Attachment #8964548 -
Flags: review?(ryanvm)
Comment 2•6 years ago
|
||
Comment on attachment 8964548 [details] [diff] [review] fix paths for MINIDUMP_STACKWALK for Windows build tasks Review of attachment 8964548 [details] [diff] [review]: ----------------------------------------------------------------- rs=me assuming you've verified that this is actually producing useful stacks now. Thanks for fixing this longstanding annoyance!
Attachment #8964548 -
Flags: review?(ryanvm) → review+
Pushed by nfroyd@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/a51cb0bc55c3 fix paths for MINIDUMP_STACKWALK for Windows build tasks; r=RyanVM
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a51cb0bc55c3
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in
before you can comment on or make changes to this bug.
Description
•