diff mbox series

data: Add exception details if build_dependencies catches one

Message ID 564bef12d9c423c71beccc5f2dfe9d50bf8ff8a2.1773815612.git.joerg.sommer@navimatix.de
State New
Headers show
Series data: Add exception details if build_dependencies catches one | expand

Commit Message

Jörg Sommer March 18, 2026, 6:33 a.m. UTC 
From: Jörg Sommer <joerg.sommer@navimatix.de>

Without this information, you'll only get messages like this, which don't
indicate what might be wrong:

    WARNING: ….bb: Exception during build_dependencies for do_patch:00:17
    WARNING: ….bb: Error during finalise of ….bb
    ERROR: ParseError in None: Not all recipes parsed, parser thread killed/died? Exiting.
    ERROR: Parsing halted due to errors, see error messages above

Signed-off-by: Jörg Sommer <joerg.sommer@navimatix.de>
---
 lib/bb/data.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jose Quaresma March 18, 2026, 9:36 a.m. UTC | #1 
Jörg Sommer via lists.openembedded.org <joerg.sommer=
navimatix.de@lists.openembedded.org> escreveu (quarta, 18/03/2026 à(s)
06:33):

> From: Jörg Sommer <joerg.sommer@navimatix.de>
>
> Without this information, you'll only get messages like this, which don't
> indicate what might be wrong:
>
>     WARNING: ….bb: Exception during build_dependencies for do_patch:00:17
>     WARNING: ….bb: Error during finalise of ….bb
>     ERROR: ParseError in None: Not all recipes parsed, parser thread
> killed/died? Exiting.
>     ERROR: Parsing halted due to errors, see error messages above
>
> Signed-off-by: Jörg Sommer <joerg.sommer@navimatix.de>
> ---
>  lib/bb/data.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/bb/data.py b/lib/bb/data.py
> index 061e63386..2eb8e0aa6 100644
> --- a/lib/bb/data.py
> +++ b/lib/bb/data.py
> @@ -367,7 +367,7 @@ def build_dependencies(key, keys, mod_funcs,
> shelldeps, varflagsexcl, ignored_va
>      except bb.parse.SkipRecipe:
>          raise
>      except Exception as e:
> -        bb.warn("Exception during build_dependencies for %s" % key)
> +        bb.warn("Exception during build_dependencies for %s: %s" % (key,
> e))
>

It was better to use 'repr(e)' Instead of assuming that 'e' will always be
a string.

Jose


>          raise
>      return frozenset(deps), value
>      #bb.note("Variable %s references %s and calls %s" % (key, str(deps),
> str(execs)))
> --
> 2.53.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#19165):
> https://lists.openembedded.org/g/bitbake-devel/message/19165
> Mute This Topic: https://lists.openembedded.org/mt/118378834/5052612
> Group Owner: bitbake-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/bitbake-devel/unsub [
> quaresma.jose@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
diff mbox series

Patch

diff --git a/lib/bb/data.py b/lib/bb/data.py
index 061e63386..2eb8e0aa6 100644
--- a/lib/bb/data.py
+++ b/lib/bb/data.py
@@ -367,7 +367,7 @@  def build_dependencies(key, keys, mod_funcs, shelldeps, varflagsexcl, ignored_va
     except bb.parse.SkipRecipe:
         raise
     except Exception as e:
-        bb.warn("Exception during build_dependencies for %s" % key)
+        bb.warn("Exception during build_dependencies for %s: %s" % (key, e))
         raise
     return frozenset(deps), value
     #bb.note("Variable %s references %s and calls %s" % (key, str(deps), str(execs)))