Hard to read error for multiconfig starting with a number

Hard to read error for multiconfig starting with a number

[jaskij]

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

When moving from Dunfell to Hardknott, I got a somewhat cryptic error
message:

```
ERROR: Unable to register event handler
'504sstate_eventhandler_stalesstate':
Traceback (most recent call last):
  File "/home/jaskij/yocto/layers/poky/meta/classes/sstate.bbclass", line 1
    def 504sstate_eventhandler_stalesstate(e):
```

As it turned out, one of my multiconfigs was named "504". Seems like
there's a function declared with a name starting with my multiconfig name.
After consulting on Matrix, RP pointed out this patch as the source:
http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=8b792d4f75b451f9e08fdb13464a0b66c11bc075

Preferred behavior would probably be adding a check and returning a
human-readable error earlier in the process.

Jan

-- 
Jan Dorniak
e-mail:jaskij@gmail.com

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

Re: [poky] Hard to read error for multiconfig starting with a number

[Richard Purdie]

On Fri, 2021-09-17 at 23:19 +0200, jaskij wrote:
> When moving from Dunfell to Hardknott, I got a somewhat cryptic error message:
> 
> ```
> ERROR: Unable to register event handler '504sstate_eventhandler_stalesstate':
> Traceback (most recent call last):
>   File "/home/jaskij/yocto/layers/poky/meta/classes/sstate.bbclass", line 1
>     def 504sstate_eventhandler_stalesstate(e):
> ```
> 
> As it turned out, one of my multiconfigs was named "504". Seems like there's a
> function declared with a name starting with my multiconfig name.
> After consulting on Matrix, RP pointed out this patch as the source:
> http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=8b792d4f75b451f9e08fdb13464a0b66c11bc075
> 
> Preferred behavior would probably be adding a check and returning a human-
> readable error earlier in the process.

Thanks for the report, I agree this should give a better error.

I've sent a patch to bitbake-devel which does that and queued
it in master-next.

Cheers,

Richard