Closed Bug 599361 Opened 14 years ago Closed 14 years ago

Stop uploading symbols to the symbol server for mobile desktop builds

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ted, Assigned: jhford)

References

Details

Attachments

(2 files)

Currently we are uploading crashreporter symbols for all the mobile desktop builds. Also, mobile builds against mozilla-central, e10s, and tracemonkey. This means we're storing 3x as many symbol files as Firefox, for builds that I would guess nobody is using as their daily browser. We should just stop uploading these symbols, I don't think they're contributing usefully to crash reporting or development. (Actual crashes on Windows/Mac/Linux are likely to be picked up in the Firefox crash reports anyway. We'd continue to get crash reports on Maemo, which is what matters, and eventually Android hopefully.)
This is to stop uploading symbols to the symbol server.
Summary: Stop uploading symbols for mobile desktop builds → Stop uploading symbols to the symbol server for mobile desktop builds
Attached patch turn off windows/mac symbols (deleted) — Splinter Review
this patch disables windows and mac symbol generation and uploading. The flags to enable/disable uploading symbols also controls whether we build them. We can change that, but will require a larger patch and testing.
If we're not uploading them, not building them doesn't matter. I think you also want to disable linux-i686 here.
you touched it, you get it!
Assignee: nobody → jhford
(In reply to comment #4) > If we're not uploading them, not building them doesn't matter. I think you also > want to disable linux-i686 here. turning it off for linux-i686 means that we stop building *and* uploading to stage, which I need for bug 511282. If we can limit scope for now to stopping windows and mac, we can land something sooner and remove 2/3 of the extraneous uploads. If we want to split building and uploading symbols, that will be a slightly larger patch.
Ah, makes sense. Seems fine.
The rest of this should have been handled in bug 607660.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: