Patchwork sanity: display the parsed BBPATH when complaining about CWD elements

login
register
mail settings
Submitter Ross Burton
Date July 19, 2012, 4:15 p.m.
Message ID <1342714534-7032-1-git-send-email-ross.burton@intel.com>
Download mbox | patch
Permalink /patch/32601/
State Accepted
Commit 6f7be2498134402b5e0f766df5c57e1f496d0d06
Headers show

Comments

Ross Burton - July 19, 2012, 4:15 p.m.
Many people don't understand the nuances of PATH/BBPATH, so help them by clarifying
the warning and displaying the parsed list element-by-element.

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/classes/sanity.bbclass |    1 +
 1 file changed, 1 insertion(+)
Saul Wold - July 21, 2012, 12:24 a.m.
On 07/19/2012 09:15 AM, Ross Burton wrote:
> Many people don't understand the nuances of PATH/BBPATH, so help them by clarifying
> the warning and displaying the parsed list element-by-element.
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>   meta/classes/sanity.bbclass |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> index 6ef5450..575ce15 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -400,6 +400,7 @@ def check_sanity(sanity_data):
>                   "Please check your layer.conf files and other BBPATH "        \
>                   "settings to remove the current working directory "           \
>                   "references.");
> +        bb.warn("Parsed BBPATH is" + str(bbpaths));
>
>       if sanity_data.getVar('TARGET_ARCH', True) == "arm":
>           # This path is no longer user-readable in modern (very recent) Linux
>

Merged into OE-Core

Thanks
	Sau!

Patch

diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 6ef5450..575ce15 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -400,6 +400,7 @@  def check_sanity(sanity_data):
                 "Please check your layer.conf files and other BBPATH "        \
                 "settings to remove the current working directory "           \
                 "references.");
+        bb.warn("Parsed BBPATH is" + str(bbpaths));
 
     if sanity_data.getVar('TARGET_ARCH', True) == "arm":
         # This path is no longer user-readable in modern (very recent) Linux