On Thu, 2011-07-14 at 21:26 +0200, Koen Kooi wrote: > The new rpath warning is nice, but a bit too terse:
Yes, agreed. Try this patch and see if the situation improves. p.
>From 5083f058ed0c559583cc9b3f4f2ae87bbbc701f9 Mon Sep 17 00:00:00 2001 From: Phil Blundell <[email protected]> Date: Fri, 15 Jul 2011 12:19:35 +0100 Subject: [PATCH] insane: improve diagnostic for redundant rpath Make it more obvious which file, and which recipe, are at issue. Signed-off-by: Phil Blundell <[email protected]> --- meta/classes/insane.bbclass | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass index c45f2cb..65eda9e 100644 --- a/meta/classes/insane.bbclass +++ b/meta/classes/insane.bbclass @@ -142,7 +142,7 @@ def package_qa_check_rpath(file,name, d, elf, messages): messages.append("package %s contains bad RPATH %s in file %s" % (name, line, file)) QAPATHTEST[useless-rpaths] = "package_qa_check_useless_rpaths" -def package_qa_check_useless_rpaths(file,name, d, elf, messages): +def package_qa_check_useless_rpaths(file, name, d, elf, messages): """ Check for RPATHs that are useless but not dangerous """ @@ -164,7 +164,7 @@ def package_qa_check_useless_rpaths(file,name, d, elf, messages): if rpath == libdir or rpath == base_libdir: # The dynamic linker searches both these places anyway. There is no point in # looking there again. - messages.append("dynamic section contains probably-redundant RPATH %s" % rpath) + messages.append("%s: %s contains probably-redundant RPATH %s" % (name, package_qa_clean_path(file, d), rpath)) QAPATHTEST[dev-so] = "package_qa_check_dev" def package_qa_check_dev(path, name, d, elf, messages): -- 1.7.4.1
_______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
