[PATCH] libtool: Add depends on binutils-cross

[PATCH] libtool: Add depends on binutils-cross

[b40527]

From: Zongchun Yu <Zongchun.Yu@freescale.com>

When use command bitbake -c populate_sdk <target-rootfs> to
generate toolchain. a race issue may occur. in this case need
to add binutils-cross to depends.

Signed-off-by: Zongchun Yu <Zongchun.Yu@freescale.com>
---
 meta/recipes-devtools/libtool/libtool-2.4.2.inc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-devtools/libtool/libtool-2.4.2.inc b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
index d26982d..0868dd7 100644
--- a/meta/recipes-devtools/libtool/libtool-2.4.2.inc
+++ b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
@@ -36,7 +36,7 @@ do_compile_prepend () {
 inherit autotools
 EXTRA_AUTORECONF = "--exclude=libtoolize"
 
-DEPENDS = "libtool-native"
+DEPENDS = "libtool-native binutils-cross"
 
 PACKAGES =+ "libltdl libltdl-dev libltdl-dbg libltdl-staticdev"
 FILES_${PN} += "${datadir}/aclocal"
-- 
1.7.0.4

Re: [PATCH] libtool: Add depends on binutils-cross

[Khem Raj]

[-- Attachment #1: Type: text/plain, Size: 355 bytes --]


On Mar 31, 2014, at 11:12 PM, <b40527@freescale.com> <b40527@freescale.com> wrote:

> From: Zongchun Yu <Zongchun.Yu@freescale.com>
> 
> When use command bitbake -c populate_sdk <target-rootfs> to
> generate toolchain. a race issue may occur. in this case need
> to add binutils-cross to depends.
> 

can you explain the race a bit here. 



[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

Re: [PATCH] libtool: Add depends on binutils-cross

[Zongchun YU]

>can you explain the race a bit here. 

Sorry to confuse you. Sometimes libtool have been built but the
binutils-cross have't been built yet.
It is required by libtool.

Re: [PATCH] libtool: Add depends on binutils-cross

[Khem Raj]

[-- Attachment #1: Type: text/plain, Size: 329 bytes --]


On Mar 31, 2014, at 11:37 PM, Zongchun YU <b40527@freescale.com> wrote:

> Sorry to confuse you. Sometimes libtool have been built but the
> binutils-cross have't been built yet.
> It is required by libtool.

OK is it cross, native or target recipe which has the problem ?
wouldnt it need to also depend on gcc-cross ?

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

Re: [PATCH] libtool: Add depends on binutils-cross

[Zongchun YU]

>OK is it cross, native or target recipe which has the problem ?
>wouldnt it need to also depend on gcc-cross ?

Only cross recipe has the problem. There is no need to gcc-cross.

Re: [PATCH] libtool: Add depends on binutils-cross

[Richard Purdie]

On Tue, 2014-04-01 at 14:12 +0800, b40527@freescale.com wrote:
> From: Zongchun Yu <Zongchun.Yu@freescale.com>
> 
> When use command bitbake -c populate_sdk <target-rootfs> to
> generate toolchain. a race issue may occur. in this case need
> to add binutils-cross to depends.
> 
> Signed-off-by: Zongchun Yu <Zongchun.Yu@freescale.com>
> ---
>  meta/recipes-devtools/libtool/libtool-2.4.2.inc |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-devtools/libtool/libtool-2.4.2.inc b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
> index d26982d..0868dd7 100644
> --- a/meta/recipes-devtools/libtool/libtool-2.4.2.inc
> +++ b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
> @@ -36,7 +36,7 @@ do_compile_prepend () {
>  inherit autotools
>  EXTRA_AUTORECONF = "--exclude=libtoolize"
>  
> -DEPENDS = "libtool-native"
> +DEPENDS = "libtool-native binutils-cross"
>  
>  PACKAGES =+ "libltdl libltdl-dev libltdl-dbg libltdl-staticdev"
>  FILES_${PN} += "${datadir}/aclocal"


You're patching the .inc file here so this dependency is being applied
to multiple different libtools. Can you perhaps explain the problem you
were seeing with a step by step how to reproduce the problem?

I'd like to understand the real problem here, I don't doubt it exists
but I'm not sure this is the correct fix.

Cheers,

Richard