[PATCH] insane.bbclass: Skip '.pc' in qa_configure

[PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Tom Rini]

There's nothing helpful in these files if they exist, and
they are often unreadable.  So lets just skip them.

[PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Tom Rini]

Signed-off-by: Tom Rini <tom_rini@mentor.com>
---
 classes/insane.bbclass |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/classes/insane.bbclass b/classes/insane.bbclass
index 5afa63c..9087efd 100644
--- a/classes/insane.bbclass
+++ b/classes/insane.bbclass
@@ -466,6 +466,8 @@ python do_qa_configure() {
     configs = []
     bb.debug(1, "Checking sanity of the config.log file")
     for root, dirs, files in os.walk(bb.data.getVar('WORKDIR', d, True)):
+        if ".pc" in root:
+            continue
         statement = "grep 'CROSS COMPILE Badness:' %s > /dev/null" % \
                     os.path.join(root,"config.log")
         if "config.log" in files:
-- 
1.6.3.3

Re: [PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Tom Rini]

Tom Rini wrote:

--compose didn't stick, grr.

These files won't have anything useful in them and are often unreadable 
so lets just skip them.

> Signed-off-by: Tom Rini <tom_rini@mentor.com>
> ---
>  classes/insane.bbclass |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/classes/insane.bbclass b/classes/insane.bbclass
> index 5afa63c..9087efd 100644
> --- a/classes/insane.bbclass
> +++ b/classes/insane.bbclass
> @@ -466,6 +466,8 @@ python do_qa_configure() {
>      configs = []
>      bb.debug(1, "Checking sanity of the config.log file")
>      for root, dirs, files in os.walk(bb.data.getVar('WORKDIR', d, True)):
> +        if ".pc" in root:
> +            continue
>          statement = "grep 'CROSS COMPILE Badness:' %s > /dev/null" % \
>                      os.path.join(root,"config.log")
>          if "config.log" in files:


-- 
Tom Rini
Mentor Graphics Corporation

Re: [PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Holger Freyther]

On 10/02/2010 05:56 AM, Tom Rini wrote:
> Tom Rini wrote:
> 
> --compose didn't stick, grr.
> 
> These files won't have anything useful in them and are often unreadable so
> lets just skip them.

s,files,files in the .pc directory,?

besides that looks like a sane change.

Re: [PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Tom Rini]

Holger Freyther wrote:
> On 10/02/2010 05:56 AM, Tom Rini wrote:
>> Tom Rini wrote:
>>
>> --compose didn't stick, grr.
>>
>> These files won't have anything useful in them and are often unreadable so
>> lets just skip them.
> 
> s,files,files in the .pc directory,?
> 
> besides that looks like a sane change.

I'll make the desc read better, thanks :)

-- 
Tom Rini
Mentor Graphics Corporation

Re: [PATCH] insane.bbclass: Skip '.pc' in qa_configure

[Khem Raj]

On Fri, Oct 1, 2010 at 2:49 PM, Tom Rini <tom_rini@mentor.com> wrote:
> Signed-off-by: Tom Rini <tom_rini@mentor.com>

Acked-by: Khem Raj <raj.khem@gmail.com>

> ---
>  classes/insane.bbclass |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/classes/insane.bbclass b/classes/insane.bbclass
> index 5afa63c..9087efd 100644
> --- a/classes/insane.bbclass
> +++ b/classes/insane.bbclass
> @@ -466,6 +466,8 @@ python do_qa_configure() {
>     configs = []
>     bb.debug(1, "Checking sanity of the config.log file")
>     for root, dirs, files in os.walk(bb.data.getVar('WORKDIR', d, True)):
> +        if ".pc" in root:
> +            continue
>         statement = "grep 'CROSS COMPILE Badness:' %s > /dev/null" % \
>                     os.path.join(root,"config.log")
>         if "config.log" in files:
> --
> 1.6.3.3
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>