414374 - Make debug mailnews builds nonstatic by default

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[[e-mail]]

...just as debug browser builds are nonlibxul by default.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Comment 2

[David :Bienvenu]

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?

Comment 3

[[e-mail]]

I'm not removing that facility, I just would like to change the default.

Comment 4

[(not currently active) Ted Mielczarek]

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.

Comment 5

[David :Bienvenu]

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.

Comment 6

[[e-mail]]

Comment on attachment 299763 [details] [diff] [review]
Proposed patch

Build config change that only affects SeaMonkey and Thunderbird debug builds.

Comment 7

[Damon Sicore (:damons)]

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

Comment 8

[[e-mail]]

(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.

Comment 9

[[e-mail]]

Need to remember to check this in after beta freeze...

Comment 10

[[e-mail]]

Fix checked in.