Right, I think the patch is ok then, but please change the commit message so that it specifically mentions UNINATIVE_LOADER.
Alex On Wed, 9 Feb 2022 at 16:27, Böszörményi Zoltán <zbos...@pr.hu> wrote: > > I guess this commit below is what broke the image build for me, > as it added UNINATIVE_LOADER handling to qemuboot.bbclass. > > commit 39c10816d5ec9d9c7952d786d7a3f942d25d0c27 > Author: Alexander Kanavin <a...@linutronix.de> > Date: Fri Feb 4 20:59:43 2022 +0100 > > runqemu: preload uninative libraries when host gl drivers are in use > > Some of the host distributions build the drivers in a way (RPATH/RUNPATH) > that tricks uninative loader into loading pieces of the host libc, if > the same pieces haven't been previously loaded by native binaries. Mixing > the two libc versions leads to failures. > > This change ensures that the correct (uninative) versions are always in > use. > > Signed-off-by: Alexander Kanavin <a...@linutronix.de> > Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org> > > > Zoltán > > 2022. 02. 09. 15:55 keltezéssel, Zoltan Boszormenyi via > lists.openembedded.org írta: > > I have added a bb.error line to print the value of "k". > > It's "UNINATIVE_LOADER". > > > > I am not touching this variable in my custom distro layer. Should I? > > > > Zoltán > > > > 2022. 02. 09. 11:56 keltezéssel, Alexander Kanavin írta: > >> Right, but this might be fixing the symptom, and not the problem. > >> > >> Alex > >> > >> On Wed, 9 Feb 2022 at 11:53, Böszörményi Zoltán <zbos...@pr.hu> wrote: > >>> > >>> (Resending from the address I am subscribed with.) > >>> > >>> I have no idea. > >>> > >>> It was working a week ago for me but today after "repo sync" > >>> the image build failed on me with the quoted error. > >>> > >>> With this fix, the image is built with the expected contents. > >>> > >>> The change with "if val.startswith()" was from 2017, > >>> something around it must have changed very recently. > >>> > >>> Zoltán > >>> > >>> 2022. 02. 09. 11:35 keltezéssel, Alexander Kanavin írta: > >>>> Wait, why val is none in the first place? Why aren't others seeing it? > >>>> > >>>> Alex > >>>> > >>>> On Wed, 9 Feb 2022 at 10:53, Zoltan Boszormenyi via > >>>> lists.openembedded.org <zboszor=pr...@lists.openembedded.org> wrote: > >>>>> > >>>>> From: Zoltán Böszörményi <zbos...@gmail.com> > >>>>> > >>>>> I got this error on current master: > >>>>> > >>>>> File: 'exec_func_python() autogenerated', lineno: 2, function: <module> > >>>>> 0001: > >>>>> *** 0002:do_write_qemuboot_conf(d) > >>>>> 0003: > >>>>> File: '.../layers/openembedded-core/meta/classes/qemuboot.bbclass', > >>>>> lineno: 141, > >>>>> function: do_write_qemuboot_conf > >>>>> 0137: else: > >>>>> 0138: val = d.getVar(k) > >>>>> 0139: # we only want to write out relative paths so that > >>>>> we can > >>>>> relocate images > >>>>> 0140: # and still run them > >>>>> *** 0141: if val.startswith(topdir): > >>>>> 0142: val = os.path.relpath(val, finalpath) > >>>>> 0143: cf.set('config_bsp', k, '%s' % val) > >>>>> 0144: > >>>>> 0145: # QB_DEFAULT_KERNEL's value of KERNEL_IMAGETYPE is the > >>>>> name of a symlink > >>>>> Exception: AttributeError: 'NoneType' object has no attribute > >>>>> 'startswith' > >>>>> > >>>>> Do nothing if "val" is None. > >>>>> > >>>>> Signed-off-by: Zoltán Böszörményi <zbos...@gmail.com> > >>>>> --- > >>>>> meta/classes/qemuboot.bbclass | 2 ++ > >>>>> 1 file changed, 2 insertions(+) > >>>>> > >>>>> diff --git a/meta/classes/qemuboot.bbclass > >>>>> b/meta/classes/qemuboot.bbclass > >>>>> index 229bd88527..755d49acd6 100644 > >>>>> --- a/meta/classes/qemuboot.bbclass > >>>>> +++ b/meta/classes/qemuboot.bbclass > >>>>> @@ -136,6 +136,8 @@ python do_write_qemuboot_conf() { > >>>>> > >>>>> 'qemu-helper-native/1.0-r1/recipe-sysroot-native/usr/bin/') > >>>>> else: > >>>>> val = d.getVar(k) > >>>>> + if val is None: > >>>>> + continue > >>>>> # we only want to write out relative paths so that we can > >>>>> relocate images > >>>>> # and still run them > >>>>> if val.startswith(topdir): > >>>>> -- > >>>>> 2.34.1 > >>>>> > >>>>> > >>>>> > >>>>> > >>> > > > > > > > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#161567): https://lists.openembedded.org/g/openembedded-core/message/161567 Mute This Topic: https://lists.openembedded.org/mt/89018458/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-