[Buildroot] [PATCH v10 8/8] Makefile: update comment about top-level parallel Makefile

[Arnout Vandecappelle <arnout@mind.be>]

On 18/12/13 11:36, Fabio Porcedda wrote:
> After the latest patches top-level parallel Makefile is working but
> there is still an issue when a package has an unspecified optional
> dependency so change the comment to explain that.
>
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> ---
>   Makefile | 15 ++++++++++++++-
>   1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index ef2582e..27a6a65 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -42,7 +42,20 @@ export HOSTARCH := $(shell uname -m | \
>   	    -e s/macppc/powerpc/\
>   	    -e s/sh.*/sh/)
>
> -# This top-level Makefile can *not* be executed in parallel
> +# Parallel execution of this Makefile is disabled because it could
> +# change the build result if a package has an unspecified optional
> +# dependency.

  ... or if two packages manipulate the same file in the target directory.

> +# Parallel execution changes the packages building order, that can be
> +# a problem when a package has an unspecified optional dependency,
> +# because if that dependency is present when the package is built, it
> +# is used, otherwise it isn't (but compilation happily proceeds). This

  This is a repeat of the first sentence.

> +# means that the packages building order is relevant in that case, and
> +# the end result will differ if the order is swapped due to parallel
> +# building.
> +# Taking in account the above warning, if you still want to execute

  Taking into acount

> +# this top-level Makefile in parrallel comment the following line and

  parallel


  Regards,
  Arnout

> +# execute:
> +#	make BR2_JLEVEL= -j$((`getconf _NPROCESSORS_ONLN`+1))
>   .NOTPARALLEL:
>
>   # absolute path
>


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F