Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@uclibc.org>
To: buildroot@busybox.net
Subject: [Buildroot] query about change to $(MAKE) definition
Date: Tue, 08 Jul 2008 10:16:58 +0200	[thread overview]
Message-ID: <87abgsu6x1.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <20080708071048.GA23482@cloud.net.au> (Hamish Moffatt's message of "Tue\, 8 Jul 2008 17\:10\:48 +1000")

>>>>> "Hamish" == Hamish Moffatt <hamish@cloud.net.au> writes:

Hi,

 Hamish> Peter,
 Hamish> Can you explain the rationale behind the following change to
 Hamish> package/Makefile.in you made last week?

Sure.

 Hamish> It's causing my kernel builds to be very noisy now. I have a
 Hamish> custom makefile for my kernel build, and I'm now getting a
 Hamish> ton of

 Hamish> make[5]: warning: -jN forced in submake: disabling jobserver mode.

Strange, I haven't seen that before.

 Hamish> warnings from it, because $(MAKE) has changed from
 Hamish> '/usr/bin/make -j2' to: '/usr/bin/make MAKE=/usr/bin/make -j2'.
 Hamish> So now -j2 is passed to every submake instance, rather than
 Hamish> letting make itself do the right thing. I'm guessing other
 Hamish> builds that do anything tricky might also suffer.

Yeah, I wanted to propagate the HOSTMAKE setting to the sub makes and
get -j and -s correct.

But now reading the make manual in details
(http://www.gnu.org/software/make/manual/html_node/Options_002fRecursion.html#Options_002fRecursion)
it seems like -s and -j are handled automatically, and from a quick
test it indeed seems to be the case for -s. Strange, that's afaik no
what I was seing before, hence the extra code.

The easiest would actually be if would get rid of the BR2_JLEVEL stuff
completely and just let people use -j<whatever> on the top level make
instead, then make should handle it all automatically. Why don't we do
it like that? Is something in the toplevel makefile broken for
parallel builds? (I know lots of other buildroot stuff is broken for
parallel). Then we could also get rid of the HOSTMAKE thing and it
would all boil down to:

MAKE1=$(MAKE) -j1

What do you say?

-- 
Bye, Peter Korsgaard

  reply	other threads:[~2008-07-08  8:16 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-08  7:10 [Buildroot] query about change to $(MAKE) definition Hamish Moffatt
2008-07-08  8:16 ` Peter Korsgaard [this message]
2008-07-08  8:26   ` Bernhard Fischer
2008-07-08  9:02     ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87abgsu6x1.fsf@macbook.be.48ers.dk \
    --to=jacmet@uclibc.org \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox