Closed
Bug 501098
Opened 15 years ago
Closed 15 years ago
Switch bloat boxes to build static mail
Categories
(Mozilla Messaging Graveyard :: Server Operations, defect)
Mozilla Messaging Graveyard
Server Operations
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
2.01 KB,
patch
|
gozer
:
review+
|
Details | Diff | Splinter Review |
We had a runtime bustage today that strangely wasn't picked up by any of the tinderboxes. I'd expected xpcshell-tests to pick it up, but they may have just got away with it. What I did realise was that the bloat boxes are building fully-shared builds. They can't do full-static builds at the moment because mac fails with a build error, but I'd at least like to add --enable-static-mail to the build options. The option builds mailnews as two dlls (one for most of mailnews, the other for import) and as a result uses slight different code. If we had had this specified, we'd have had a chance of picking up the runtime bustage because we'd have been hitting the appropriate code. The non-static mail option is really only maintained for developers, so lets switch this and get the code a bit nearer to what we actually release.
Assignee | ||
Comment 1•15 years ago
|
||
This adds --enable-static-mail to all the bloat mozconfigs. I've also made debug/optimise be consistently ordered across the files. Probably best also to clobber when we check this due to the changes from multiple to a few dlls.
Comment 2•15 years ago
|
||
Comment on attachment 385727 [details] [diff] [review] The fix This is fine, and doesn't require a buildbot reconfiguration. Just land this and the next set of bloat builds will pick up the new mozconfigs.
Attachment #385727 -
Flags: review?(gozer) → review+
Assignee | ||
Comment 3•15 years ago
|
||
Landed and clobbered builds: http://hg.mozilla.org/build/buildbot-configs/rev/3a576e4435da Currently waiting for them to cycle.
Assignee | ||
Comment 4•15 years ago
|
||
This was fixed a while ago.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•