476149 - make unit tests work again on comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Robert Kaiser]

comm-central build have stopped doing unit tests properly while still calling them correctly from mozilla/

This happened since bug 475111 landed.

Comment 1

[Robert Kaiser]

Attachment: set TIERS before calling config.mk, WIP

The basic problem seems to be that in the main Makefile the subdirectories to be called are set through TIERS and tier_*_dirs which are actually set in build.mk.
As long as rules.mk was converting TIERS to DIRS, that worked fine with the current placement of that include, but since I landed the port of bug 466486, this is done in config.mk already, so we need to move the build.mk inclusion before the one for config.mk now. autoconf.mk needs to be included even before that or MOZ_BUILD_APP isn't set.

I'm running into a problem in testing this right now, suddenly normal "make" wants to enter the installer/ subdir and breaks.

Comment 2

[Robert Kaiser]

Attachment: set TIERS before calling config.mk

We also need to set an empty default target at the beginning of the Makefile or we enter the "installer" target that happens to be the first defined one (in build.mk).

This all closely mirrors what the main Mozilla Makefile is doing, by the way.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 359740 [details] [diff] [review]
set TIERS before calling config.mk

r=me based on inspection, lets see how this goes.

Comment 4

[Robert Kaiser]

Pushed as http://hg.mozilla.org/comm-central/rev/1360769469ea