[oe] [PATCH] arptables: defeat multi-job for install

Paul Eggleton paul.eggleton at linux.intel.com
Sat Feb 16 20:31:11 UTC 2013


Hi Joe,

On Thursday 14 February 2013 13:40:04 Joe Slater wrote:
> CQID: 399299
> 
> The make install target is not parallel job
> safe, but doesn't do all that much, so we
> limit make to one job for that.
> 
> The make target is of the form
> 
> install: x y z
> 
> Unfortunately, z depends on y, so if z is started
> before y (almost) completes, we will fail.
> 
> Signed-off-by: Joe Slater <jslater at windriver.com>
> ---
>  recipes-support/arptables/arptables_0.0.3-4.bb |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/recipes-support/arptables/arptables_0.0.3-4.bb
> b/recipes-support/arptables/arptables_0.0.3-4.bb index 2faccec..de3f62d
> 100644
> --- a/recipes-support/arptables/arptables_0.0.3-4.bb
> +++ b/recipes-support/arptables/arptables_0.0.3-4.bb
> @@ -3,6 +3,8 @@ PRIORITY = "optional"
>  LICENSE = "GPL-2.0"
>  SECTION = "console/network"
> 
> +PR = "r2"
> +
>  RDEPENDS_${PN} += "perl"
> 
>  LIC_FILES_CHKSUM =
> "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6
> 425610833a22dbe6" @@ -20,6 +22,11 @@ do_compile () {
>  	oe_runmake
>  }
> 
> +# the install target is not multi-job safe, but it doesn't do much
> +# so we just install seriatim
> +#

I hope I don't expose my ignorance too much by saying I just learned a new 
word thanks to the above comment... ;)

> +PARALLEL_MAKEINST = "-j1"

Of course this will work but typically we would just set PARALLEL_MAKEINST = 
"" to accomplish this.

More importantly, this appears to be a patch for the meta-networking layer and 
should therefore have [meta-networking] in the subject and use the meta-
networking/ path.

Cheers,
Paul




More information about the Openembedded-devel mailing list