[PATCH] kernel-yocto: Fix bb.debug to add missing log level

[PATCH] kernel-yocto: Fix bb.debug to add missing log level

[Richard Purdie]

Fixes warnings like:

WARNING: linux-intel-5.4.50+gitAUTOINC+caafbdfe38_89e6c056f3-r0 do_kernel_configcheck: Passed invalid debug level 'outfile size is: /home/pokybuild/yocto-worker/meta-intel/build/build/tmp/work-shared/intel-corei7-64/kernel-source/.kernel-meta/cfg/mismatch.txt 767' to bb.debug

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/classes/kernel-yocto.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/kernel-yocto.bbclass b/meta/classes/kernel-yocto.bbclass
index 3c428c0782e..7f870a6cc61 100644
--- a/meta/classes/kernel-yocto.bbclass
+++ b/meta/classes/kernel-yocto.bbclass
@@ -515,7 +515,7 @@ python do_kernel_configcheck() {
         with open(outfile, 'w+') as f:
             f.write( analysis )
 
-        bb.debug( "outfile size is: %s %s" % (outfile,os.stat(outfile).st_size))
+        bb.debug(1, "outfile size is: %s %s" % (outfile,os.stat(outfile).st_size))
         if config_check_visibility and os.stat(outfile).st_size > 0:
             with open (outfile, "r") as myfile:
                 results = myfile.read()
-- 
2.25.1

✗ patchtest: failure for kernel-yocto: Fix bb.debug to add missing log level

[Patchwork]

== Series Details ==

Series: kernel-yocto: Fix bb.debug to add missing log level
Revision: 1
URL   : https://patchwork.openembedded.org/series/25494/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue             Series does not apply on top of target branch [test_series_merge_on_head] 
  Suggested fix    Rebase your series on top of targeted branch
  Targeted branch  master (currently at 10221e578f)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

Re: [OE-core] [PATCH] kernel-yocto: Fix bb.debug to add missing log level

[Bruce Ashfield]

[-- Attachment #1: Type: text/plain, Size: 1717 bytes --]

On Thu, Aug 6, 2020 at 5:15 AM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> Fixes warnings like:
>
> WARNING: linux-intel-5.4.50+gitAUTOINC+caafbdfe38_89e6c056f3-r0
> do_kernel_configcheck: Passed invalid debug level 'outfile size is:
> /home/pokybuild/yocto-worker/meta-intel/build/build/tmp/work-shared/intel-corei7-64/kernel-source/.kernel-meta/cfg/mismatch.txt
> 767' to bb.debug
>
>
Argh.

Apparently I didn't hit any mismatches after putting that debug statement
in. I was getting a file with a single \n and it was breaking the size
test, hence the debug addition.

Thanks for the fixup on this one.

Cheers,

Bruce



> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta/classes/kernel-yocto.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/kernel-yocto.bbclass
> b/meta/classes/kernel-yocto.bbclass
> index 3c428c0782e..7f870a6cc61 100644
> --- a/meta/classes/kernel-yocto.bbclass
> +++ b/meta/classes/kernel-yocto.bbclass
> @@ -515,7 +515,7 @@ python do_kernel_configcheck() {
>          with open(outfile, 'w+') as f:
>              f.write( analysis )
>
> -        bb.debug( "outfile size is: %s %s" %
> (outfile,os.stat(outfile).st_size))
> +        bb.debug(1, "outfile size is: %s %s" %
> (outfile,os.stat(outfile).st_size))
>          if config_check_visibility and os.stat(outfile).st_size > 0:
>              with open (outfile, "r") as myfile:
>                  results = myfile.read()
> --
> 2.25.1
>
> 
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end
- "Use the force Harry" - Gandalf, Star Trek II

[-- Attachment #2: Type: text/html, Size: 3070 bytes --]