[Buildroot] [PATCH 1/2] Makefile: toolchain-eclipse-register: add toolchain dependency

[Buildroot] [PATCH 1/2] Makefile: toolchain-eclipse-register: add toolchain dependency

[Fabio Porcedda]

The "toolchain-eclipse-register" target needs the toolchain so add
toolchain as a dependency.
This also fix the support to top-level parallel make.

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 42adf7c..53282ae 100644
--- a/Makefile
+++ b/Makefile
@@ -621,7 +621,7 @@ target-post-image: $(TARGETS_ROOTFS) target-finalize
 		$(call MESSAGE,"Executing post-image script $(s)"); \
 		$(EXTRA_ENV) $(s) $(BINARIES_DIR) $(call qstrip,$(BR2_ROOTFS_POST_SCRIPT_ARGS))$(sep))
 
-toolchain-eclipse-register:
+toolchain-eclipse-register: toolchain
 	./support/scripts/eclipse-register-toolchain `readlink -f $(O)` $(notdir $(TARGET_CROSS)) $(BR2_ARCH)
 
 source: $(TARGETS_SOURCE) $(HOST_SOURCE)
-- 
1.9.2

[Buildroot] [PATCH 2/2] Makefile: target-generatelocales: add toolchain dependency

[Fabio Porcedda]

The target depends on the toolchain so add it as a dependency.
This also fix the support to top-level parallel make.

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 53282ae..e204038 100644
--- a/Makefile
+++ b/Makefile
@@ -598,7 +598,7 @@ ifneq ($(GENERATE_LOCALE),)
 # data comes preferably from the toolchain, or if the toolchain does
 # not have them (Linaro toolchains), we use the ones available on the
 # host machine.
-target-generatelocales: host-localedef
+target-generatelocales: host-localedef toolchain
 	$(Q)mkdir -p $(TARGET_DIR)/usr/lib/locale/
 	$(Q)for locale in $(GENERATE_LOCALE) ; do \
 		inputfile=`echo $${locale} | cut -f1 -d'.'` ; \
-- 
1.9.2

[Buildroot] [PATCH 2/2] Makefile: target-generatelocales: add toolchain dependency

[Fabio Porcedda]

On Fri, Apr 18, 2014 at 3:07 PM, Fabio Porcedda
<fabio.porcedda@gmail.com> wrote:
> The target depends on the toolchain so add it as a dependency.
> This also fix the support to top-level parallel make.
>
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> ---
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 53282ae..e204038 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -598,7 +598,7 @@ ifneq ($(GENERATE_LOCALE),)
>  # data comes preferably from the toolchain, or if the toolchain does
>  # not have them (Linaro toolchains), we use the ones available on the
>  # host machine.
> -target-generatelocales: host-localedef
> +target-generatelocales: host-localedef toolchain
>         $(Q)mkdir -p $(TARGET_DIR)/usr/lib/locale/
>         $(Q)for locale in $(GENERATE_LOCALE) ; do \
>                 inputfile=`echo $${locale} | cut -f1 -d'.'` ; \
> --
> 1.9.2
>

I've done like that because i think that the "target-generatelocales"
depends only on the toolchain target, doesn't it?

Regards
-- 
Fabio Porcedda

[Buildroot] [PATCH 1/2] Makefile: toolchain-eclipse-register: add toolchain dependency

[Thomas Petazzoni]

Dear Fabio Porcedda,

On Fri, 18 Apr 2014 15:07:31 +0200, Fabio Porcedda wrote:
> The "toolchain-eclipse-register" target needs the toolchain so add
> toolchain as a dependency.
> This also fix the support to top-level parallel make.
> 
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> ---
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Both patches applied.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com