Patchwork [Consolidated,Pull,23/23] mdadm: add binutils to DEPENDS

login
register
mail settings
Submitter Saul Wold
Date June 17, 2012, 5:40 a.m.
Message ID <b7bfb9b693cc592f081a3f42ef3f874a937d305d.1339911478.git.sgw@linux.intel.com>
Download mbox | patch
Permalink /patch/30037/
State New
Headers show

Comments

Saul Wold - June 17, 2012, 5:40 a.m.
mdadm requires ansidecl.h and therefore binutils needs to be completely
installed before this compile can complete, so add the DEPENDS.

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-extended/mdadm/mdadm_3.2.5.bb |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
Richard Purdie - June 17, 2012, 8:59 a.m.
On Sat, 2012-06-16 at 22:40 -0700, Saul Wold wrote:
> mdadm requires ansidecl.h and therefore binutils needs to be completely
> installed before this compile can complete, so add the DEPENDS.
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta/recipes-extended/mdadm/mdadm_3.2.5.bb |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
> index 2d7e1d0..244777b 100644
> --- a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
> +++ b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
> @@ -9,6 +9,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>                      file://mdadm.h;beglinlne=4;endline=22;md5=462bc9936ac0d3da110191a3f9994161"
>  
>  PR = "r0"
> +DEPENDS = "binutils" 
>  
>  SRC_URI = "${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.bz2 \
>  	    file://mdadm-3.2.2_fix_for_x32.patch \

I think we can likely just remove the use of the header:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=37cf162998b13d7d594c7ed68c47d440a3033ed5

Cheers,

Richard
Saul Wold - June 18, 2012, 5:35 a.m.
On 06/17/2012 01:59 AM, Richard Purdie wrote:
> On Sat, 2012-06-16 at 22:40 -0700, Saul Wold wrote:
>> mdadm requires ansidecl.h and therefore binutils needs to be completely
>> installed before this compile can complete, so add the DEPENDS.
>>
>> Signed-off-by: Saul Wold<sgw@linux.intel.com>
>> ---
>>   meta/recipes-extended/mdadm/mdadm_3.2.5.bb |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
>> index 2d7e1d0..244777b 100644
>> --- a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
>> +++ b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
>> @@ -9,6 +9,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>>                       file://mdadm.h;beglinlne=4;endline=22;md5=462bc9936ac0d3da110191a3f9994161"
>>
>>   PR = "r0"
>> +DEPENDS = "binutils"
>>
>>   SRC_URI = "${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.bz2 \
>>   	    file://mdadm-3.2.2_fix_for_x32.patch \
>
> I think we can likely just remove the use of the header:
>
> http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=37cf162998b13d7d594c7ed68c47d440a3033ed5
>
Fixed in the branch oe-contrib/sgw/stage branch as suggested.

Sau!
> Cheers,
>
> Richard
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>

Patch

diff --git a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
index 2d7e1d0..244777b 100644
--- a/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
+++ b/meta/recipes-extended/mdadm/mdadm_3.2.5.bb
@@ -9,6 +9,7 @@  LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
                     file://mdadm.h;beglinlne=4;endline=22;md5=462bc9936ac0d3da110191a3f9994161"
 
 PR = "r0"
+DEPENDS = "binutils" 
 
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.bz2 \
 	    file://mdadm-3.2.2_fix_for_x32.patch \