If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Stop stripping mac nightlies

RESOLVED FIXED

Status

Thunderbird
Build Config
P2
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: gozer, Assigned: standard8)

Tracking

Trunk
x86
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
+++ This bug was initially created as a clone of Bug #515645 +++

We should --disable-install-strip our mac nightlies.  That will let us shark them, make the apple crash reporter work (in cases when breakpad doesn't trigger for some reason), and generally make them happier campers.

The claim is that the download size increase here is negligible, which I buy.
(Assignee)

Comment 1

8 years ago
Created attachment 403205 [details] [diff] [review]
[checked in] The fix

Ok, this sounds a reasonable thing to do. If I understand correctly the major benefits for us are:

1) Using the apple profiler should give us correct symbols for nightly builds. This would be helpful in the case of hangs.
2) If crash reporter failed, then the apple crash reporter would pick up the right symbols.

The downside is a larger nightly build. Looking at Firefox's changes here's some important additional notes:

- Firefox has not done this for release builds (this patch should also not do it for release builds, but I'd like gozer to verify that).

- There was an approx 6Mb increase in the size of Mac nightlies on Firefox when stripping was disabled. The partial mar was 8Mb that day.

- Subsequent partial mars for the next few days were typically a 200 to 300 kb.

Although the build size is generally seen as something to be kept small, I think we should take this hit as I've seen various situations recently (especially with our hangs on shutdown) where it would be useful for nightly users to have this symbol information.
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #403205 - Flags: superreview?(dmose)
Attachment #403205 - Flags: review?(gozer)
(Reporter)

Comment 2

8 years ago
Comment on attachment 403205 [details] [diff] [review]
[checked in] The fix

And Standard8 is correct, this will not affect release builds.
Attachment #403205 - Flags: review?(gozer) → review+
A 6mb increase from what to what?

Will making this change to the nightlies also mean that actual releases will get it also?
To be clear, I'm asking about dmg size differences, not mar size differences.
(Assignee)

Comment 5

8 years ago
(In reply to comment #3)
> A 6mb increase from what to what?

The .dmg size. From before the change (which was stripping symbols) to after the change (not stripping symbols).

> Will making this change to the nightlies also mean that actual releases will
> get it also?

No. See my comment and gozer's comments.
(In reply to comment #5)
> (In reply to comment #3)
> > A 6mb increase from what to what?
> 
> The .dmg size. From before the change (which was stripping symbols) to after
> the change (not stripping symbols).

Right, but what was the size and the size after?
Er size before and size after.  

/me swigs some more coffee.
(Assignee)

Comment 8

8 years ago
For Firefox (from 1.9.1): 17405 KB -> 23668 KB

Thunderbird's current nightly size is 19081 KB

Updated

8 years ago
Attachment #403205 - Flags: superreview?(dmose) → superreview+
Comment on attachment 403205 [details] [diff] [review]
[checked in] The fix

sr=dmose
(Reporter)

Updated

8 years ago
Attachment #403205 - Attachment description: The fix → [checked in] The fix
(Reporter)

Comment 10

8 years ago
Comment on attachment 403205 [details] [diff] [review]
[checked in] The fix

comparing with ssh://hg.mozilla.org/build/buildbot-configs
searching for changes
changeset:   1553:90d473dda32b
tag:         tip
user:        Philippe M. Chiasson <gozer@mozillamessaging.com>
date:        Mon Sep 28 15:16:58 2009 -0400
summary:     Bug 518175 - Stop stripping mac nightlies. r=gozer, sr=dmose
(Reporter)

Comment 11

8 years ago
Buildbot mater reconfig-ed. Tonight's nightlies should run with this mozconfig
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.