Benjamin Smedberg Assignee
	Description • 16 years ago

While researching build time, I discovered that we're using a subshell to recurse over subdirectories even when there aren't any. This is wasteful, especially on Windows where processes cost a lot.

	(not currently active) Ted Mielczarek
	Comment 1 • 16 years ago

Comment on attachment 349760 [details] [diff] [review]
Avoid shells for recursing empty DIRS, rev. 1

Do we ever wind up with DIRS set to an empty space? Couldn't we just use |ifdef DIRS| here?

Also, amazing what silliness you can find with a little research!

	Benjamin Smedberg Assignee
	Comment 2 • 16 years ago

I found at least one case with something like:

DIRS += \
  $(EMPTYVALUE) \
  $(NULL)

oh, for make variables that are lists and not scalars ;-)

	Benjamin Smedberg Assignee
	Comment 3 • 16 years ago

http://hg.mozilla.org/mozilla-central/rev/f71446b6fc7e

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 4 • 16 years ago

this broke x86_64/linux builds

	Benjamin Smedberg Assignee
	Comment 5 • 16 years ago

backed out... weird regressions with some directories not getting built

	neil@parkwaycc.co.uk
	Comment 6 • 16 years ago

Comment on attachment 349760 [details] [diff] [review]
Avoid shells for recursing empty DIRS, rev. 1

>-	@$(EXIT_ON_ERROR) \
>-	$(foreach dir,$(STATIC_DIRS),$(MAKE) -C $(dir); ) true
>+	+$(LOOP_OVER_STATIC_DIRS)

>-ifdef TOOL_DIRS
>-	@$(EXIT_ON_ERROR) \
>-	$(foreach dir,$(TOOL_DIRS),$(UPDATE_TITLE) $(MAKE) -C $(dir) libs; ) true
>-endif
>+	+$(LOOP_OVER_TOOL_DIRS)

Unfortunately $(LOOP_OVER_XXX_DIRS) uses $@ and so these aren't the same...

	Benjamin Smedberg Assignee
	Comment 7 • 16 years ago

In this version, you aren't allowed to modify DIRS after you've included rules.mk... it would have already caused problems, such as missing submakefile dependencies. Unfortunately I can't think of a good way to enforce this rule, but oh well...

This also fixes misuse of LOOP_OVER_STATIC_DIRS and LOOP_OVER_TOOL_DIRS

	Phil Ringnalda (:philor)
	Comment 8 • 16 years ago

Looks like both browser/Makefile.in and mail/Makefile.in muck with DIRS after including rules.mk.

	Benjamin Smedberg Assignee
	Comment 9 • 16 years ago

That turns out not to be a problem in practice (because DIRS was already set to something), but I've patched it.

	Benjamin Smedberg Assignee
	Comment 10 • 16 years ago

Second time's the charm: http://hg.mozilla.org/mozilla-central/rev/d11ccfc1d754

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 11 • 16 years ago

Comment on attachment 353702 [details] [diff] [review]
Avoid shells for recursing empty DIRS, rev. 2.1

a191=beltzner

	Benjamin Smedberg Assignee
	Comment 12 • 16 years ago

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/758a1804c901