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)

enhancement
Not set
normal

Tracking

(firefox61 fixed)

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

References

Details

Attachments

(1 file)

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.
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
https://hg.mozilla.org/mozilla-central/rev/a51cb0bc55c3
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: