[PATCH 1/1] ltp: find all .debug directories

[PATCH 1/1] ltp: find all .debug directories

[Joe Slater]

The list of directories for ltp-dbg is incomplete, so
we generalize it.

We also eliminate a non-fatal qa error that the file
test_arch_stripped is stripped.

Signed-off-by: Joe Slater <jslater@windriver.com>
---
 meta/recipes-extended/ltp/ltp_20150119.bb |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-extended/ltp/ltp_20150119.bb b/meta/recipes-extended/ltp/ltp_20150119.bb
index b422646..90c5973 100644
--- a/meta/recipes-extended/ltp/ltp_20150119.bb
+++ b/meta/recipes-extended/ltp/ltp_20150119.bb
@@ -65,9 +65,9 @@ RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect"
 
 FILES_${PN}-dbg += "\
     /opt/ltp/runtest/.debug \
-    /opt/ltp/testcases/bin/.debug \
-    /opt/ltp/testcases/bin/*/bin/.debug \
-    /opt/ltp/testcases/bin/*/test/.debug \
+    /opt/ltp/testcases/*/.debug \
+    /opt/ltp/testcases/*/*/.debug \
+    /opt/ltp/testcases/*/*/*/.debug \
     /opt/ltp/scenario_groups/.debug \
     /opt/ltp/testscripts/.debug \
     /opt/ltp/testscripts/open_posix_testsuite/.debug \
@@ -79,3 +79,6 @@ FILES_${PN} += "/opt/ltp/* /opt/ltp/runtest/* /opt/ltp/scenario_groups/* /opt/lt
 
 # Avoid generated binaries stripping. Otherwise some of the ltp tests such as ldd01 & nm01 fails
 INHIBIT_PACKAGE_STRIP = "1"
+# However, test_arch_stripped is already stripped, so...
+INSANE_SKIP_${PN} += "already-stripped"
+
-- 
1.7.9.5

Re: [PATCH 1/1] ltp: find all .debug directories

[Khem Raj]

On Wed, Mar 18, 2015 at 3:34 PM, Joe Slater <jslater@windriver.com> wrote:
>  INHIBIT_PACKAGE_STRIP = "1"
> +# However, test_arch_stripped is already stripped, so...
> +INSANE_SKIP_${PN} += "already-stripped"
> +

Can this be avoided in the component make system instead

Re: [PATCH 1/1] ltp: find all .debug directories

[Slater, Joseph]

> -----Original Message-----
> From: Khem Raj [mailto:raj.khem@gmail.com]
> Sent: Wednesday, March 18, 2015 4:37 PM
> To: Slater, Joseph
> Cc: Patches and discussions about the oe-core layer
> Subject: Re: [OE-core] [oe-core][PATCH 1/1] ltp: find all .debug directories
> 
> On Wed, Mar 18, 2015 at 3:34 PM, Joe Slater <jslater@windriver.com> wrote:
> >  INHIBIT_PACKAGE_STRIP = "1"
> > +# However, test_arch_stripped is already stripped, so...
> > +INSANE_SKIP_${PN} += "already-stripped"
> > +
> 
> Can this be avoided in the component make system instead

Do you mean not strip it when we build it?  I hesitate to have some named _stripped
not be.  Since nobody seems upset by the current qa "error", I guess we could also
just ignore it.

Joe

Re: [PATCH 1/1] ltp: find all .debug directories

[Khem Raj]

> On Mar 19, 2015, at 9:44 AM, Slater, Joseph <joe.slater@windriver.com> wrote:
> 
> 
> 
>> -----Original Message-----
>> From: Khem Raj [mailto:raj.khem@gmail.com]
>> Sent: Wednesday, March 18, 2015 4:37 PM
>> To: Slater, Joseph
>> Cc: Patches and discussions about the oe-core layer
>> Subject: Re: [OE-core] [oe-core][PATCH 1/1] ltp: find all .debug directories
>> 
>> On Wed, Mar 18, 2015 at 3:34 PM, Joe Slater <jslater@windriver.com> wrote:
>>> INHIBIT_PACKAGE_STRIP = "1"
>>> +# However, test_arch_stripped is already stripped, so...
>>> +INSANE_SKIP_${PN} += "already-stripped"
>>> +
>> 
>> Can this be avoided in the component make system instead
> 
> Do you mean not strip it when we build it?


yes

>  I hesitate to have some named _stripped
> not be.

?

>  Since nobody seems upset by the current qa "error", I guess we could also
> just ignore it.

I think it can cause debugging gripes. So its good to fix it

> 
> Joe
> 

Re: [PATCH 1/1] ltp: find all .debug directories

[Burton, Ross]

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

On 19 March 2015 at 00:34, Joe Slater <jslater@windriver.com> wrote:

>  FILES_${PN}-dbg += "\
>      /opt/ltp/runtest/.debug \
> -    /opt/ltp/testcases/bin/.debug \
> -    /opt/ltp/testcases/bin/*/bin/.debug \
> -    /opt/ltp/testcases/bin/*/test/.debug \
> +    /opt/ltp/testcases/*/.debug \
> +    /opt/ltp/testcases/*/*/.debug \
> +    /opt/ltp/testcases/*/*/*/.debug \
>

Interestingly now ltp is failing on edgerouter (mips64):

NOTE: Skipping file /opt/ltp/testcases/bin/test_arch_stripped from ltp for
already-stripped QA test
ERROR: debugedit failed with exit code 256 (cmd was
'/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/sysroots/x86_64-linux/usr/lib/rpm/bin/debugedit'
-b
'/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/work/mips64-poky-linux'
-d '/usr/src/debug' -i -l
'/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/work/mips64-poky-linux/ltp/20150119-r0/debugsources.list'
'/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/work/mips64-poky-linux/ltp/20150119-r0/package/opt/ltp/testcases/bin/d1.obj'):
/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/sysroots/x86_64-linux/usr/lib/rpm/bin/debugedit:
/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-mips/build/build/tmp/work/mips64-poky-linux/ltp/20150119-r0/package/opt/ltp/testcases/bin/d1.obj:
Unhandled relocation 2 in .debug_info section

I'm off to bed now, any idea what "relocation 2 in ._debug_info" means?

Ross

[-- Attachment #2: Type: text/html, Size: 2508 bytes --]