Make debug mailnews builds nonstatic by default

RESOLVED FIXED

Status

enhancement
RESOLVED FIXED
12 years ago
Last year

People

(Reporter: neil, Assigned: neil)

Tracking

Trunk

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

...just as debug browser builds are nonlibxul by default.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #299763 - Flags: superreview?(bienvenu)
Attachment #299763 - Flags: review?(ted.mielczarek)
My fear about this is that when people add components, they'll forget the static build and break tinderbox. I think mailnews probably adds components more frequently than libxul does.

It would be nice if each developer could decide what they wanted with a .mozconfig option - that's not possible now, is it?

I'm not removing that facility, I just would like to change the default.
Comment on attachment 299763 [details] [diff] [review]
Proposed patch

This should be fine, we do the same thing with libxul, since it's painful to build debug.  We trust that developers will watch the tinderbox for bustage if they broke something in libxul only.
Attachment #299763 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 299763 [details] [diff] [review]
Proposed patch

OK - this should be announced pretty widely, and it would be really nice as part of that announcement to tell people how to change their mozconfig to keep the previous behavior.
Attachment #299763 - Flags: superreview?(bienvenu) → superreview+
Comment on attachment 299763 [details] [diff] [review]
Proposed patch

Build config change that only affects SeaMonkey and Thunderbird debug builds.
Attachment #299763 - Flags: approval1.9?
Comment on attachment 299763 [details] [diff] [review]
Proposed patch


What exactly are you going to do to address David's concerns above? Let's make sure it happens. 

a1.9+=damons
Attachment #299763 - Flags: approval1.9? → approval1.9+
(In reply to comment #7)
>What exactly are you going to do to address David's concerns above?
I've topic'd #maildev and announced in three newsgroups.
Need to remember to check this in after beta freeze...
Keywords: checkin-needed
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.