summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKevin Hao <kexin.hao@windriver.com>2015-09-30 10:29:33 +0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2015-10-01 07:40:37 +0100
commit4965f122ca67c0ff60dc60f7885db1ed9db909b4 (patch)
tree709d4d56fba8e04c6aa7f32f2160866f033a1ecd
parente8e4cf4e830ee5c5f92dd8ab38c4072a3a43c411 (diff)
downloadopenembedded-core-contrib-4965f122ca67c0ff60dc60f7885db1ed9db909b4.tar.gz
openembedded-core-contrib-4965f122ca67c0ff60dc60f7885db1ed9db909b4.tar.bz2
openembedded-core-contrib-4965f122ca67c0ff60dc60f7885db1ed9db909b4.zip
kernel.bbclass: fix the bug of checking the existing sections in do_strip()
Ross reported the following waring when building edgerouter BSP: WARNING: Section not found: .comment The reason is that the testing of the existing sections in do_strip() returned the wrong value. Please see the following code in do_strip(): for str in ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS}; do { if [ "$headers" != *"$str"* ]; then bbwarn "Section not found: $str"; fi "$CROSS_COMPILE"strip -s -R $str ${KERNEL_OUTPUT} }; done The "*" doesn't have special meaning in the if string test, so it will return true even the $str is a substring of $headers. Fix this issue by replacing it with "! (echo "$headers" | grep -q "^$str$")". Reported-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Kevin Hao <kexin.hao@windriver.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r--meta/classes/kernel.bbclass2
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index dfbdfd24ff..5e8b6cf343 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -413,7 +413,7 @@ do_strip() {
gawk '{print $1}'`
for str in ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS}; do {
- if [ "$headers" != *"$str"* ]; then
+ if ! (echo "$headers" | grep -q "^$str$"); then
bbwarn "Section not found: $str";
fi