Re: [OE-core] [PATCH 1/1] insane.bbclass: Added QA test for expanded ${D}

2015-01-13 Thread Alejandro Hernandez
Thank Ross, sending new patch soon. On 07/01/15 12:27, Burton, Ross wrote: Hi Alejandro, Looks good, but some small points: On 11 December 2014 at 22:40, Alejandro Hernandez > wrote: - version-going-backwards \ +versio

Re: [OE-core] [PATCH 1/1] insane.bbclass: Added QA test for expanded ${D}

2015-01-07 Thread Burton, Ross
On 7 January 2015 at 22:02, Randy Witt wrote: > Would it be simpler to do "bbvar = d.getVar(var + "_" + pak, False)" so > you don't get the expanded value, and then just check for ${D}? I suppose > it would save one extra variable and a couple of expansions. > Alejandro answered this point in th

Re: [OE-core] [PATCH 1/1] insane.bbclass: Added QA test for expanded ${D}

2015-01-07 Thread Randy Witt
On 12/11/2014 02:40 PM, Alejandro Hernandez wrote: Checks in FILES and pkg_* variables, solves common mistake of using ${D} instead of $D and warns the user accordingly. [YOCTO #6642] Signed-off-by: Alejandro Hernandez --- meta/classes/insane.bbclass | 30 +- 1

Re: [OE-core] [PATCH 1/1] insane.bbclass: Added QA test for expanded ${D}

2015-01-07 Thread Burton, Ross
Hi Alejandro, Looks good, but some small points: On 11 December 2014 at 22:40, Alejandro Hernandez < alejandro.hernan...@linux.intel.com> wrote: > -version-going-backwards \ > +version-going-backwards expanded_d \ > +QAPATHTEST[expanded_d] = "package_qa_check_expanded_d"

[OE-core] [PATCH 1/1] insane.bbclass: Added QA test for expanded ${D}

2014-12-11 Thread Alejandro Hernandez
Checks in FILES and pkg_* variables, solves common mistake of using ${D} instead of $D and warns the user accordingly. [YOCTO #6642] Signed-off-by: Alejandro Hernandez --- meta/classes/insane.bbclass | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --g