Patchwork [v3,1/3] package/Makefile.in: add a way to don't force jobs in sub-make

login
register
mail settings
Submitter Fabio Porcedda
Date Sept. 10, 2013, 7:46 a.m.
Message ID <1378799164-28645-2-git-send-email-fabio.porcedda@gmail.com>
Download mbox | patch
Permalink /patch/273969/
State Accepted
Commit 4b3bc153ad78615d71a307998d432e79d5d32cf4
Headers show

Comments

Fabio Porcedda - Sept. 10, 2013, 7:46 a.m.
When the "BR2_JLEVEL" variable is empty use "make"
without the "-j" option.
To be able to use top-level parallel make we must don't force
the number of jobs in sub-make.

Example:
	make BR2_JLEVEL= -j8

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Thomas Petazzoni - Sept. 12, 2013, 6:28 p.m.
Dear Fabio Porcedda,

On Tue, 10 Sep 2013 09:46:02 +0200, Fabio Porcedda wrote:
> When the "BR2_JLEVEL" variable is empty use "make"
> without the "-j" option.
> To be able to use top-level parallel make we must don't force
> the number of jobs in sub-make.
> 
> Example:
> 	make BR2_JLEVEL= -j8
> 
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Patch

diff --git a/package/Makefile.in b/package/Makefile.in
index 91ae1b9..6702df4 100644
--- a/package/Makefile.in
+++ b/package/Makefile.in
@@ -18,7 +18,7 @@  PARALLEL_JOBS:=$(BR2_JLEVEL)
 endif
 
 MAKE1:=$(HOSTMAKE) -j1
-MAKE:=$(HOSTMAKE) -j$(PARALLEL_JOBS)
+MAKE:=$(HOSTMAKE) $(if $(PARALLEL_JOBS),-j$(PARALLEL_JOBS))
 
 # Compute GNU_TARGET_NAME
 GNU_TARGET_NAME=$(ARCH)-buildroot-$(TARGET_OS)-$(LIBC)$(ABI)