Closed Bug 465329 Opened 11 years ago Closed 11 years ago

sync to breakpad revision 303

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: ted, Assigned: ted)

References

Details

(Keywords: fixed1.9.1)

Attachments

(3 files, 1 obsolete file)

We should sync up to the latest Breakpad code to pick up Mac DWARF support.
Summary: sync to breakpad revision 294 → sync to breakpad revision 296
Blocks: 445090
Blocks: 423745
I have a patch for just the Breakpad parts, but it's 2.7Mb. I'll have to figure out a decent way of presenting it here. There are some follow-up fixes necessary to Mozilla code to go along with this, but I'll break them out into a separate patch for ease of reviewing.

There are a few weird bits in the patch, notably:
- Lots of file permissions changed (I originally imported files on a Win32 machine, I did this import on a Linux machine, so the permissions are not all goofy)
- Some files have their line endings changed from Windows to Unix. Also a result of the above, should be harmless.

The rest of the patch is all Breakpad changes, including adding a bunch of new files.
Summary: sync to breakpad revision 296 → sync to breakpad revision 297
This is `grep -A2 "^diff"` on my patch. This will at least let you see how many of the file changes are just mode changes, how many are new files, etc.
Attached patch mozilla changes (obsolete) — Splinter Review
These are the changes needed on our side to build properly with the updated code. I've built this on all three platforms locally, but not on the exact same code, so I've pushed this to the try server to double check.
Comment on attachment 350008 [details]
sync to r299: just the filenames

Really just looking for an rs here, but if you'd like more details I can post the full patch for you.
Attachment #350008 - Flags: review?(benjamin)
Comment on attachment 350033 [details] [diff] [review]
mozilla changes

All green on the try server.
Attachment #350033 - Flags: review?(benjamin)
Attached patch mozilla changesSplinter Review
Slight addition, the dwarf library requires -funsigned-char to work correctly.
Attachment #350033 - Attachment is obsolete: true
Attachment #350670 - Flags: review?(benjamin)
Attachment #350033 - Flags: review?(benjamin)
Bumping up the revision, I checked in a fix (and a fix to my fix). The list of filenames from the diff doesn't change, but the patch contents do slightly.
http://code.google.com/p/google-breakpad/source/detail?r=298
http://code.google.com/p/google-breakpad/source/detail?r=299
Summary: sync to breakpad revision 297 → sync to breakpad revision 299
Attachment #350670 - Flags: review?(benjamin) → review+
Attachment #350008 - Flags: review?(benjamin) → review+
Attachment #350008 - Attachment description: just the filenames → sync to r299: just the filenames
This is just a diff from r299 - r303. These changes have all been reviewed upstream. This finishes the changes I need to make DWARF support work for us on Mac.
Attachment #351883 - Flags: review?(benjamin)
Attachment #351883 - Flags: review?(benjamin) → review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/7aa964f60934
http://hg.mozilla.org/mozilla-central/rev/681b46e69db2
http://hg.mozilla.org/mozilla-central/rev/31281697c166
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Summary: sync to breakpad revision 299 → sync to breakpad revision 303
We should land this and bug 421534 on 1.9.1 after it bakes on trunk for a while.
Comment on attachment 350008 [details]
sync to r299: just the filenames

I'd like to land all three of these patches on the 1.9.1 branch, along with the changes in bug 421534. (And two tiny followup fixes.)

I know this looks like a big scary set of patches, but the net effect is really very tiny, localized mostly to the production of debug symbols on Mac. However, the benefit is pretty big, since this will let us get rid of -save-temps from the CFLAGS of our Mac build machines, which will save us several gigs per build tree in space. (and make RelEng quite happy). In addition, since Thunderbird is now building off of 1.9.1, this should fix a few bad bugs that Thunderbird hits with symbol dumping (since they build static).
Attachment #350008 - Flags: approval1.9.1?
Attachment #350008 - Flags: approval1.9.1? → approval1.9.1+
You need to log in before you can comment on or make changes to this bug.