kmod: fix debuginfo is missing in shared library

The following changes since commit 924da20c6f2f183fa7e930e4c3c6b14c090acbe5:

build-appliance-image: Update to dizzy head revision (2014-10-18 16:16:38 +0200)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib chonglu/kmod
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=chonglu/kmod

Chong Lu (1):
kmod: fix debuginfo is missing in shared library

 meta/recipes-kernel/kmod/kmod_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

1.9.1

The INHIBIT_PACKAGE_STRIP should be only for ptest package, else debuginfo will be missing in shared library.

---
 meta/recipes-kernel/kmod/kmod_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

index d4c21a4..ff58153 100644
@@ -44,7 +44,7 @@ do_compile_ptest () {
         oe_runmake buildtest-TESTS rootfs
 }
 
-INHIBIT_PACKAGE_STRIP = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", "1", "0", d)}"
+INHIBIT_PACKAGE_STRIP_${PN}-ptest = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", "1", "0", d)}"
 INSANE_SKIP_${PN}-ptest = "arch"
 
 inherit update-alternatives

1.9.1

Was this actually tested?

ERROR: runstrip: ''arm-poky-linux-gnueabi-strip' --strip-debug
--remove-section=.comment --remove-section=.note --preserve-dates '/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-arm/build/build/tmp/work/armv5te-poky-linux-gnueabi/kmod/18+gitAUTOINC+ae58de0fcb-r0/package/usr/lib/kmod/ptest/testsuite/rootfs/test-depmod/detect-loop/lib/modules/4.4.4/kernel/moduleA.ko'' strip command failed

INHIBIT_PACKAGE_STRIP isn't a per-package variable as stripping happens
before the package-splitting. Moving it to happen after splitting won't
be trivial as you'd need to know what package to put the resultant files into. Maybe we need a strip blacklist?

Ross

Hi,

From: [email protected] [mailto:[email protected]] On Behalf Of Burton, Ross
Sent: Thursday, October 23, 2014 17:21
To: Chong Lu
Cc: OE-core
Subject: Re: [OE-core] [PATCH 1/1] kmod: fix debuginfo is missing in shared library

On 21 October 2014 09:16, Chong Lu <[email protected]<mailto:[email protected]>> wrote:
-INHIBIT_PACKAGE_STRIP = "${@bb.utils.contains(<mailto:$%[email protected](>"DISTRO_FEATURES", "ptest", "1", "0", d)}" +INHIBIT_PACKAGE_STRIP_${PN}-ptest = "${@bb.utils.contains(<mailto:$%[email protected](>"DISTRO_FEATURES", "ptest", "1", "0", d)}"

Was this actually tested?

ERROR: runstrip: ''arm-poky-linux-gnueabi-strip' --strip-debug --remove-section=.comment --remove-section=.note --preserve-dates '/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-arm/build/build/tmp/work/armv5te-poky-linux-gnueabi/kmod/18+gitAUTOINC+ae58de0fcb-r0/package/usr/lib/kmod/ptest/testsuite/rootfs/test-depmod/detect-loop/lib/modules/4.4.4/kernel/moduleA.ko'' strip command failed

INHIBIT_PACKAGE_STRIP isn't a per-package variable as stripping happens before the package-splitting. Moving it to happen after splitting won't be trivial as you'd need to know what package to put the resultant files into. Maybe we need a strip blacklist?

Ross

One way to solve this is to pack the ptest “sample” files during install (in do_install_append) and unpack back when running ptest on target (in run-ptest). The downside of this is we add a dependency on tar. Please let me know if it is acceptable and I’ll come with a patch for this.
Regards,
Tudor.

Drop this one, I have send a v2.

Best Regards
Chong

Thanks for your suggestion, I have sent a v2.

Best Regards
Chong