I see, thanks for the clarification. After looking into this I can say that in fact
the issue was caused by this commit:
<https://git.yoctoproject.org/meta-virtualization/commit/?id=c02af94731631b3432b8ddee2b89761ab32530ee>


It fails because meta-intel missed to set the patch version in LINUX_VERSION in linux-intel_6.10.bb
<https://git.yoctoproject.org/meta-intel/tree/recipes-kernel/linux/linux-intel_6.10.bb#n16>

changing to LINUX_VERSION ?= "6.10.0" fixes the issue

On Thu, Sep 12 2024 at 11:32:52 AM +02:00:00, Martin Jansa <[email protected]> wrote:
That's weird, these LINUX_MAJOR/LINUX_MINOR are used only to include
the .inc file in:
recipes-kernel/linux/linux-%.bbappend:include
${@bb.utils.contains <mailto:${@bb.utils.contains>('DISTRO_FEATURES', 'virtualization',
'linux-${KERNEL_META_TYPE}_${LINUX_MAJOR}.${LINUX_MINOR}_virtualization.inc',
'', d)}

while yocto-cfg-fragments dependency you've mentioned in the error is
added based on PV now in this .inc file:
recipes-kernel/linux/linux-yocto_virtualization.inc:        return
"yocto-cfg-fragments-%s-native:do_populate_sysroot" % pv

So I think your fix only prevents either of these to be included:
./recipes-kernel/linux/linux-yocto_virtualization.inc
./recipes-kernel/linux/linux-yocto_6.6_virtualization.inc
./recipes-kernel/linux/linux-yocto_6.10_virtualization.inc

and not only for people using meta-intel, but for everybody, because
there is no linux-yocto_x.y_virtualization.inc at all.

That doesn't look right. I think
"oe.utils.trim_version(d.getVar('PV'), 2)" should be improved to strip
trailing +git as well or maybe to use LINUX_MAJOR/LINUX_MINOR instead
of PV inside the .inc file.

Regards,

On Thu, Sep 12, 2024 at 10:48 AM Markus Volk <[email protected] <mailto:[email protected]>> wrote:

On Thu, Sep 12 2024 at 10:36:04 AM +02:00:00, Martin Jansa <[email protected] <mailto:[email protected]>> wrote:

Wasn't this fixed by <https://git.yoctoproject.org/meta-virtualization/commit/?id=c02af94731631b3432b8ddee2b89761ab32530ee> ?

I have meta-virtualization layer up to date, so it includes this commit, but was still hit by this problem.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8890): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8890
Mute This Topic: https://lists.yoctoproject.org/mt/108409634/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to