Re: [PATCH] uki.bbclass: fix ukify arguments assembly

["Pavel Löbl" <pavel@loebl.cz>]

On Thu, Apr 02, 2026 at 10:52:11AM +0300, Mikko Rapeli wrote:
> Hi,
> 
> On Thu, Apr 02, 2026 at 09:33:15AM +0200, Pavel L�bl wrote:
> > Missing space before --uname ukify option mangles the command line
> > string when KERNEL_VERSION is set.
> 
> Thanks, this is a clear bug.
> 
> meta/lib/oeqa/selftest/cases/uki.py could be improved to cover this.
> I would be interested to know if the kernel version autodetection
> does not work since that is preferred by ukify.py and this KERNEL_VERSION
> handling in uki.bbclass could be removed.
It works for me now. I just ran into this bug while trying different class
variables. In the end I had build misconfigured in other places (missing efi
machine feature and possible some other stuff).

Regards,
Pavel

> 
> Reviewed-by: Mikko Rapeli <mikko.rapeli@linaro.org>
> 
> Cheers,
> 
> -Mikko
> 
> > Signed-off-by: Pavel L�bl <pavel@loebl.cz>
> > ---
> >  meta/classes-recipe/uki.bbclass | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/classes-recipe/uki.bbclass b/meta/classes-recipe/uki.bbclass
> > index 326697a06660..a89e92b77869 100644
> > --- a/meta/classes-recipe/uki.bbclass
> > +++ b/meta/classes-recipe/uki.bbclass
> > @@ -140,7 +140,7 @@ python do_uki() {
> >          # not always needed, ukify can detect version from kernel binary
> >          kernel_version = d.getVar('KERNEL_VERSION')
> >          if kernel_version:
> > -            ukify_cmd += "--uname %s" % (kernel_version)
> > +            ukify_cmd += " --uname %s" % (kernel_version)
> >      else:
> >          bb.fatal("ERROR - UKI_KERNEL_FILENAME not set")
> >  
> > -- 
> > 2.53.0
> >