[PATCH 0/1] package.bbclass: create debugsources.list if it doesn't exist

[PATCH 0/1] package.bbclass: create debugsources.list if it doesn't exist

[Robert Yang]

The testing information:
# Take MACHINE=qemux86 as example:
1) $ bitbake core-image-sato
2) $ find tmp/work/i586-poky-linux/ -maxdepth 3 -name 'log.do*' -exec \
        grep -H 'debugsources.list: No such file or directory' {} \;

   # There should be no output(more than 100 lines without this patch).

// Robert

The following changes since commit f47ada62a3da879006e7cb27479dc9b72c56e923:

  cmake.bbclass: add ${base_libdir} to CMAKE_LIBRARY_PATH (2012-03-12 17:46:16 +0000)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib robert/debugsources
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/debugsources

Robert Yang (1):
  package.bbclass: create debugsources.list if it doesn't exist

 meta/classes/package.bbclass |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

-- 
1.7.4.1

[PATCH 1/1] package.bbclass: create debugsources.list if it doesn't exist

[Robert Yang]

Serval pkg's debug pkg is null, so the find-debuginfo.sh would not
generate the ${WORKDIR}/debugsources.list, but the processdebugsrc
always need it, there are a few such errors currently:(53 pkgs have such
errors in a core-image-sato build):

log.do_package:sort: open failed: debugsources.list: No such file or directory

Create debugsources.list if it doesn't exist would fix this problem.

[YOCTO #2076]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/classes/package.bbclass |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 9b6862d..8e76ef8 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -243,6 +243,11 @@ def splitfile2(debugsrcdir, d):
     workparentdir = os.path.dirname(workdir)
     workbasedir = os.path.basename(workdir)
     sourcefile = d.expand("${WORKDIR}/debugsources.list")
+    # There is no debugsources.list when the debug pkg is null, but the
+    # processdebugsrc needs it, so create one if it doesn't exist.
+    if not os.path.isfile(sourcefile):
+        f = open(sourcefile, 'w')
+        f.close()
 
     if debugsrcdir:
        nosuchdir = []
-- 
1.7.4.1

Re: [PATCH 1/1] package.bbclass: create debugsources.list if it doesn't exist

[Richard Purdie]

On Tue, 2012-03-13 at 10:50 +0800, Robert Yang wrote:
> Serval pkg's debug pkg is null, so the find-debuginfo.sh would not
> generate the ${WORKDIR}/debugsources.list, but the processdebugsrc
> always need it, there are a few such errors currently:(53 pkgs have such
> errors in a core-image-sato build):
> 
> log.do_package:sort: open failed: debugsources.list: No such file or directory
> 
> Create debugsources.list if it doesn't exist would fix this problem.

Why can't we modify the code that uses debugsources.list not to get
upset if it doesn't exist?

This change seems a little backwards to me...

Cheers,

Richard

> [YOCTO #2076]
> 
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
>  meta/classes/package.bbclass |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index 9b6862d..8e76ef8 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -243,6 +243,11 @@ def splitfile2(debugsrcdir, d):
>      workparentdir = os.path.dirname(workdir)
>      workbasedir = os.path.basename(workdir)
>      sourcefile = d.expand("${WORKDIR}/debugsources.list")
> +    # There is no debugsources.list when the debug pkg is null, but the
> +    # processdebugsrc needs it, so create one if it doesn't exist.
> +    if not os.path.isfile(sourcefile):
> +        f = open(sourcefile, 'w')
> +        f.close()
>  
>      if debugsrcdir:
>         nosuchdir = []

Re: [PATCH 1/1] package.bbclass: create debugsources.list if it doesn't exist

[Robert Yang]

On 03/14/2012 01:15 AM, Richard Purdie wrote:
> On Tue, 2012-03-13 at 10:50 +0800, Robert Yang wrote:
>> Serval pkg's debug pkg is null, so the find-debuginfo.sh would not
>> generate the ${WORKDIR}/debugsources.list, but the processdebugsrc
>> always need it, there are a few such errors currently:(53 pkgs have such
>> errors in a core-image-sato build):
>>
>> log.do_package:sort: open failed: debugsources.list: No such file or directory
>>
>> Create debugsources.list if it doesn't exist would fix this problem.
>
> Why can't we modify the code that uses debugsources.list not to get
> upset if it doesn't exist?
>

Sounds great! I will send an update patch sooner.

Will I reply the updated patch here or should I send a V2, please?
I don't know which is popular:-).

// Robert


> This change seems a little backwards to me...
>
> Cheers,
>
> Richard
>
>> [YOCTO #2076]
>>
>> Signed-off-by: Robert Yang<liezhi.yang@windriver.com>
>> ---
>>   meta/classes/package.bbclass |    5 +++++
>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
>> index 9b6862d..8e76ef8 100644
>> --- a/meta/classes/package.bbclass
>> +++ b/meta/classes/package.bbclass
>> @@ -243,6 +243,11 @@ def splitfile2(debugsrcdir, d):
>>       workparentdir = os.path.dirname(workdir)
>>       workbasedir = os.path.basename(workdir)
>>       sourcefile = d.expand("${WORKDIR}/debugsources.list")
>> +    # There is no debugsources.list when the debug pkg is null, but the
>> +    # processdebugsrc needs it, so create one if it doesn't exist.
>> +    if not os.path.isfile(sourcefile):
>> +        f = open(sourcefile, 'w')
>> +        f.close()
>>
>>       if debugsrcdir:
>>          nosuchdir = []
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>