Patchwork [PATH] fix build failure due to lic_files_chksum mismatch. The correction unpacks the source in the correct directory.

login
mail settings
Submitter Iyad Qumei
Date July 6, 2012, 11:02 p.m.
Message ID <1341615750-25579-1-git-send-email-iyadkq@gmail.com>
Download mbox | patch
Permalink /patch/31353/
State New
Headers show

Comments

Iyad Qumei - July 6, 2012, 11:02 p.m.
Signed-off-by: Iyad Qumei <iyadkq@gmail.com>
---
 meta/recipes-core/netbase/netbase_5.0.bb |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
Martin Jansa - July 7, 2012, 2:31 a.m.
On Fri, Jul 06, 2012 at 04:02:30PM -0700, Iyad Qumei wrote:
> Signed-off-by: Iyad Qumei <iyadkq@gmail.com>
> ---
>  meta/recipes-core/netbase/netbase_5.0.bb |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-core/netbase/netbase_5.0.bb b/meta/recipes-core/netbase/netbase_5.0.bb
> index 3aea874..ad80ef6 100644
> --- a/meta/recipes-core/netbase/netbase_5.0.bb
> +++ b/meta/recipes-core/netbase/netbase_5.0.bb
> @@ -4,7 +4,7 @@ HOMEPAGE = "http://packages.debian.org/netbase"
>  SECTION = "base"
>  LICENSE = "GPLv2"
>  LIC_FILES_CHKSUM = "file://debian/copyright;md5=3dd6192d306f582dee7687da3d8748ab"
> -PR = "r0"
> +PR = "r1"
>  
>  inherit update-rc.d
>  
> @@ -17,6 +17,8 @@ SRC_URI = "${DEBIAN_MIRROR}/main/n/netbase/netbase_${PV}.tar.gz \
>             file://interfaces \
>             file://nfsroot"
>  
> +S = "${WORKDIR}/${P}"
> +

This is default, so it shouldn't change anything (and it checks
lic_files fine here).

Cheers,

>  SRC_URI[md5sum] = "a379d9199d44d817543eddc211a29b70"
>  SRC_URI[sha256sum] = "ac9424d3cb2bd3c07b6c5bc5875c434d3bc687a73cd7c54b7dc7122308afa9dc"
>  
> -- 
> 1.7.4.1
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Iyad Qumei - July 9, 2012, 3:39 a.m.
>
> This is default, so it shouldn't change anything (and it checks
> lic_files fine here).

in my case, the error is caused by the fact the oe-core is setting up
the work-directory under i586-xx ( the architecture area ) while the
unpacking was done in qemux86-xxx ( the machine area ).  The only way
to fix it was to explicitly define the S= in the recipe.

Thanks for the feedback.
Iyad

On Fri, Jul 6, 2012 at 7:31 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> On Fri, Jul 06, 2012 at 04:02:30PM -0700, Iyad Qumei wrote:
>> Signed-off-by: Iyad Qumei <iyadkq@gmail.com>
>> ---
>>  meta/recipes-core/netbase/netbase_5.0.bb |    4 +++-
>>  1 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/recipes-core/netbase/netbase_5.0.bb b/meta/recipes-core/netbase/netbase_5.0.bb
>> index 3aea874..ad80ef6 100644
>> --- a/meta/recipes-core/netbase/netbase_5.0.bb
>> +++ b/meta/recipes-core/netbase/netbase_5.0.bb
>> @@ -4,7 +4,7 @@ HOMEPAGE = "http://packages.debian.org/netbase"
>>  SECTION = "base"
>>  LICENSE = "GPLv2"
>>  LIC_FILES_CHKSUM = "file://debian/copyright;md5=3dd6192d306f582dee7687da3d8748ab"
>> -PR = "r0"
>> +PR = "r1"
>>
>>  inherit update-rc.d
>>
>> @@ -17,6 +17,8 @@ SRC_URI = "${DEBIAN_MIRROR}/main/n/netbase/netbase_${PV}.tar.gz \
>>             file://interfaces \
>>             file://nfsroot"
>>
>> +S = "${WORKDIR}/${P}"
>> +
>
> This is default, so it shouldn't change anything (and it checks
> lic_files fine here).
>
> Cheers,
>
>>  SRC_URI[md5sum] = "a379d9199d44d817543eddc211a29b70"
>>  SRC_URI[sha256sum] = "ac9424d3cb2bd3c07b6c5bc5875c434d3bc687a73cd7c54b7dc7122308afa9dc"
>>
>> --
>> 1.7.4.1
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
Khem Raj - July 9, 2012, 4:45 a.m.
On Sun, Jul 8, 2012 at 8:39 PM, Iyad Qumei <iyadkq@gmail.com> wrote:
>>
>> This is default, so it shouldn't change anything (and it checks
>> lic_files fine here).
>
> in my case, the error is caused by the fact the oe-core is setting up
> the work-directory under i586-xx ( the architecture area ) while the
> unpacking was done in qemux86-xxx ( the machine area ).

interesting scenario. Recipe does use MACHINE variable during
do_install which essentially makes this recipe
machine specific so  I would expect it to build completely under the
machinedir. Case you are explaining above
is strange. I wonder if you have some bbappend which is changing some
properties.

 The only way
> to fix it was to explicitly define the S= in the recipe.
>

the patch is not needed since S is set to WORKDIR/P by default.

> Thanks for the feedback.
> Iyad
>
> On Fri, Jul 6, 2012 at 7:31 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
>> On Fri, Jul 06, 2012 at 04:02:30PM -0700, Iyad Qumei wrote:
>>> Signed-off-by: Iyad Qumei <iyadkq@gmail.com>
>>> ---
>>>  meta/recipes-core/netbase/netbase_5.0.bb |    4 +++-
>>>  1 files changed, 3 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/meta/recipes-core/netbase/netbase_5.0.bb b/meta/recipes-core/netbase/netbase_5.0.bb
>>> index 3aea874..ad80ef6 100644
>>> --- a/meta/recipes-core/netbase/netbase_5.0.bb
>>> +++ b/meta/recipes-core/netbase/netbase_5.0.bb
>>> @@ -4,7 +4,7 @@ HOMEPAGE = "http://packages.debian.org/netbase"
>>>  SECTION = "base"
>>>  LICENSE = "GPLv2"
>>>  LIC_FILES_CHKSUM = "file://debian/copyright;md5=3dd6192d306f582dee7687da3d8748ab"
>>> -PR = "r0"
>>> +PR = "r1"
>>>
>>>  inherit update-rc.d
>>>
>>> @@ -17,6 +17,8 @@ SRC_URI = "${DEBIAN_MIRROR}/main/n/netbase/netbase_${PV}.tar.gz \
>>>             file://interfaces \
>>>             file://nfsroot"
>>>
>>> +S = "${WORKDIR}/${P}"
>>> +
>>
>> This is default, so it shouldn't change anything (and it checks
>> lic_files fine here).
>>
>> Cheers,
>>
>>>  SRC_URI[md5sum] = "a379d9199d44d817543eddc211a29b70"
>>>  SRC_URI[sha256sum] = "ac9424d3cb2bd3c07b6c5bc5875c434d3bc687a73cd7c54b7dc7122308afa9dc"
>>>
>>> --
>>> 1.7.4.1
>>>
>>>
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>> --
>> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>
> _______________________________________________
> 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-core/netbase/netbase_5.0.bb b/meta/recipes-core/netbase/netbase_5.0.bb
index 3aea874..ad80ef6 100644
--- a/meta/recipes-core/netbase/netbase_5.0.bb
+++ b/meta/recipes-core/netbase/netbase_5.0.bb
@@ -4,7 +4,7 @@  HOMEPAGE = "http://packages.debian.org/netbase"
 SECTION = "base"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://debian/copyright;md5=3dd6192d306f582dee7687da3d8748ab"
-PR = "r0"
+PR = "r1"
 
 inherit update-rc.d
 
@@ -17,6 +17,8 @@  SRC_URI = "${DEBIAN_MIRROR}/main/n/netbase/netbase_${PV}.tar.gz \
            file://interfaces \
            file://nfsroot"
 
+S = "${WORKDIR}/${P}"
+
 SRC_URI[md5sum] = "a379d9199d44d817543eddc211a29b70"
 SRC_URI[sha256sum] = "ac9424d3cb2bd3c07b6c5bc5875c434d3bc687a73cd7c54b7dc7122308afa9dc"