72018 - depend builds don't pick up changes to config.mk

Status: VERIFIED WONTFIX

(SeaMonkey :: Build Config, defect)

Description

[Bradley Baetz]

Last night rods checked in a patch to config.mk to turn some debugging stuff on
by default. This wasn't picked up as a dependancy change, so when someone
changed a file in layout, we ended up with two separate vtables (as some files
were recompiled and some weren't) for nsIPresContext -> core dump on startup.

The corresponding config.mak change didn't cause problems for win32, according
to tinderbox. They may just not build --enable-debug - I don't know what their
setup is.
mcaffee clobbered coffee last night, and leaf just did harpoon, which should
clear the alive test problems (The bloat test errors look different)

Comment 1

[cls]

Known problem.  No reasonable fix except don't put or change defines in
config.mk.  Put them in configure.in.  If you put config.mk as a dependency of
the source files, then every tweak to config.mk will cause the entire tree to
rebuild.  And most changes to config.mk do not require rebuilding hardly any
parts of the tree.  

Comment 2

[Jon Granrose]

verified wontfix.