Closed Bug 1496405 Opened 11 months ago Closed 11 months ago

mozcrash doesn't do the right thing on Windows when Python is 64-bit and Firefox is 32-bit

Categories

(Testing :: Mozbase, defect)

Version 3
defect
Not set

Tracking

(firefox64 fixed)

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: ted, Assigned: cmanchester)

References

Details

Attachments

(1 file)

bug 1194935 added a minidumpwriter binary so that when we were using a 32-bit Python but building a 64-bit Firefox we could write minidumps of the Firefox process. The logic in the moz.build only builds that program for win64:
https://dxr.mozilla.org/mozilla-central/rev/dcba2a476ccfb77f859b3614d2c77ed3cc06a225/testing/tools/minidumpwriter/moz.build#7

The mozcrash logic is written to look for it when the Python is a different word size than Firefox:
https://dxr.mozilla.org/mozilla-central/rev/dcba2a476ccfb77f859b3614d2c77ed3cc06a225/testing/mozbase/mozcrash/mozcrash/mozcrash.py#418

As it turns out, in bug 1413630 we updated our Windows builders to MozillaBuild 3.1, which includes a 64-bit Python, so now minidumpwriter *isn't* used for 64-bit builds, but that conditional trips for 32-bit builds and the binary is missing. I noticed this while looking at a log from bug 1408980:
https://treeherder.mozilla.org/logviewer.html#?job_id=202385111&repo=try&lineNumber=97049

15:45:05     INFO - E   mozcrash Using z:\build\build\src\obj-firefox\dist\bin\minidumpwriter.exe to write a dump to c:\users\task_1538229602\appdata\local\temp\xpc-other-pn6uef\ea9b5363-665c-453b-83e7-027ab9ae58e3.dmp for [4552]
15:45:05     INFO - E   mozcrash minidumpwriter not found in z:/build/build/src/obj-firefox\dist\bin

I'm filing this under mozcrash, but I guess alternately we could just change the moz.build logic to always build this binary on Windows, or to build it just for 32-bit Firefox since it's mostly useful for CI anyway.
Just building if for windows everywhere seems to be the easiest and least fragile thing here...
Assignee: nobody → cmanchester
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8ce6780e0f5
Always build minidumpwriter on Windows to support 64 or 32 bit python. r=ted,firefox-build-system-reviewers
https://hg.mozilla.org/mozilla-central/rev/d8ce6780e0f5
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.