Stop uploading symbols for test programs/libs to the symbol server

RESOLVED FIXED in mozilla16

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: ted, Assigned: ted)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
These aren't useful for anything and they're currently wasting 50GB of space. We're going to rm the ones that are there, but we should stop uploading them too.
(Assignee)

Updated

5 years ago
Assignee: nobody → ted.mielczarek
(Assignee)

Comment 1

5 years ago
I wrote a patch that skips *test* and *Test* files, and it works, but there's still a fair bit of junk:
BadPlugin
GoodPlugin
ReadNTLM
ShowAlignments
ShowSSEConfig
SimpleTypeLib
WriteArgument
certutiljs
mangle
modutil
nsIFileEnumerator
nsinstall
pk12util
rdfcat
rdfpoll
shlibsign
ssltunnel
triplescat

I guess I can change it to be an exclusions file and we can just list most of this stuff there. Kind of sucks, but it'd help a bit.
(Assignee)

Comment 2

5 years ago
Created attachment 633231 [details] [diff] [review]
Stop uploading symbols for test programs/libs to the symbol server

Here's what I've got so far, including unit tests. This drops $(DIST)/crashreporter-symbols from 35MB to 31MB on my Mac.
(Assignee)

Comment 3

5 years ago
Comment on attachment 633231 [details] [diff] [review]
Stop uploading symbols for test programs/libs to the symbol server

Feel like reviewing this, Nick?
Attachment #633231 - Flags: review?(nrthomas)
Comment on attachment 633231 [details] [diff] [review]
Stop uploading symbols for test programs/libs to the symbol server

Looks fine to me.
Attachment #633231 - Flags: review?(nrthomas) → review+
(Assignee)

Comment 5

5 years ago
I pushed this to try along with my patch for bug 587073, so I'll wait to make sure that doesn't break anything first.
(Assignee)

Comment 6

5 years ago
The ideal solution for this would be to only dump symbols for things that wind up in the package, but we have to run buildsymbols before make package to make everything work right. If we rejiggered things a bit so that we had the preprocessed package manifest available we could use that as a whitelist. That's probably a good follow-up.
Comment on attachment 633231 [details] [diff] [review]
Stop uploading symbols for test programs/libs to the symbol server

I meant to ask if the tests will be run in 'make check' or equivalent ? Otherwise they only test the native platform when run.
d'oh, that would be the hunk in toolkit/crashreporter/Makefile.in.
(Assignee)

Comment 9

5 years ago
Yeah, they get run during "make check", and I've verified that they pass on Mac/Win/Linux.
(Assignee)

Comment 10

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/602cfef02b3d
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/602cfef02b3d
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in before you can comment on or make changes to this bug.