Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
On Wed, May 29, 2024 at 5:34 AM ChenQi wrote: > > Hi Bruce, > > I did some investigation this afternoon. A patch series has been sent, > which mostly comes from 'grep + sed', and also includes a patch from Martin. > I did build tests, the result looks good to me. I think the only > remaining recipes that need to be dealt with are: yq and possibly runv. runv has been put into read-only mode, so I'm going to remove it from the layer and pull in a replacement. So nothing really to do on their front. Bruce > I'll need to do something else tomorrow. Hope you or someone else could > check more. > Hope this would help a little bit. > > Regards, > Qi > > On 5/28/24 04:37, Bruce Ashfield via lists.yoctoproject.org wrote: > > As an update. My patience is wearing thin. > > > > The posted patches don't build for me, my changes don't build for me, > > if I hack things, the runtime is broken. > > > > I'm seeing different fetching and source management based on the > > number of git repositories in the SCR_URI, which means I'm running > > into everything from missing source to license checksum failures when > > I reorganize the code. > > > > I'm stubborn about this sort of thing, there's no sense in me just > > merging patches that don't build on my machines, and/or patches that I > > don't understand how they fix issues, since I'll never be able to > > maintain things afterwards. > > > > I am still working on this, but I'm no debugging oe-core code to try > > and understand what is happening and why the behaviour is changing, so > > everything is extremely hacked up and I'm not even near a RFT branch > > yet. > > > > Summary: it will be quite some time before I have meta-virt working > > again, since I feel like I've had a bag of rocks dumped right on top > > of me and changes to core that I don't have straight in my head yet. > > > > Bruce > > > > On Fri, May 24, 2024 at 3:40 PM Martin Jansa wrote: > >> On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield > >> wrote: > >>> Interesting. > >>> > >>> I've been through the layer now and have done all the simple semantic > >>> changes to adjust things. > >>> > >>> I'm almost to runtime testing, but expect that by Monday I'll have > >>> everything sorted. > >> OK, looking forward to the master-next update (and kirkstone update) > >> to have a bit greener world builds again :). > >> > >>> Bruce > >>> > >>> On Fri, May 24, 2024 at 3:26 PM Martin Jansa > >>> wrote: > It was failing yesterday, but today it built OK, I didn't do any > runtime testing (only checked buildhistory-diff). > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > still failing for me. > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > wrote: > > Did you get this to build with just that change ? > > > > My patches to cni are significantly larger to get it to build and > > create binaries. > > > > Bruce > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > lists.yoctoproject.org > > wrote: > >> * see: > >> > >> https://lists.openembedded.org/g/openembedded-architecture/message/2007 > >> > >> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > >> > >> * fixes: > >>http://errors.yoctoproject.org/Errors/Details/772691/ > >>ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > >> TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > >> [license-checksum] > >> > >> Signed-off-by: Martin Jansa > >> --- > >> recipes-networking/cni/cni_git.bb | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/recipes-networking/cni/cni_git.bb > >> b/recipes-networking/cni/cni_git.bb > >> index b41b47cf..c0f316ec 100644 > >> --- a/recipes-networking/cni/cni_git.bb > >> +++ b/recipes-networking/cni/cni_git.bb > >> @@ -14,7 +14,7 @@ SRCREV_plugins = > >> "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > >> SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > >> SRCREV_FORMAT = "cni_plugins" > >> SRC_URI = "\ > >> - > >> git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > >> \ > >> + > >> git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > >> \ > >> > >> git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > >> \ > >> > >> git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel >
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
FWIW: my colleague sent me some fixes for my WIP jansa/master branch in: https://github.com/shr-project/meta-virtualization/pull/1 https://github.com/shr-project/meta-virtualization/pull/2 I haven't shared them as I'm waiting for master-next branch to be updated to see what's left to fix (and hard to guess what Bruce is cooking) On Wed, May 29, 2024 at 11:34 AM ChenQi wrote: > > Hi Bruce, > > I did some investigation this afternoon. A patch series has been sent, > which mostly comes from 'grep + sed', and also includes a patch from Martin. > I did build tests, the result looks good to me. I think the only > remaining recipes that need to be dealt with are: yq and possibly runv. > I'll need to do something else tomorrow. Hope you or someone else could > check more. > Hope this would help a little bit. > > Regards, > Qi > > On 5/28/24 04:37, Bruce Ashfield via lists.yoctoproject.org wrote: > > As an update. My patience is wearing thin. > > > > The posted patches don't build for me, my changes don't build for me, > > if I hack things, the runtime is broken. > > > > I'm seeing different fetching and source management based on the > > number of git repositories in the SCR_URI, which means I'm running > > into everything from missing source to license checksum failures when > > I reorganize the code. > > > > I'm stubborn about this sort of thing, there's no sense in me just > > merging patches that don't build on my machines, and/or patches that I > > don't understand how they fix issues, since I'll never be able to > > maintain things afterwards. > > > > I am still working on this, but I'm no debugging oe-core code to try > > and understand what is happening and why the behaviour is changing, so > > everything is extremely hacked up and I'm not even near a RFT branch > > yet. > > > > Summary: it will be quite some time before I have meta-virt working > > again, since I feel like I've had a bag of rocks dumped right on top > > of me and changes to core that I don't have straight in my head yet. > > > > Bruce > > > > On Fri, May 24, 2024 at 3:40 PM Martin Jansa wrote: > >> On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield > >> wrote: > >>> Interesting. > >>> > >>> I've been through the layer now and have done all the simple semantic > >>> changes to adjust things. > >>> > >>> I'm almost to runtime testing, but expect that by Monday I'll have > >>> everything sorted. > >> OK, looking forward to the master-next update (and kirkstone update) > >> to have a bit greener world builds again :). > >> > >>> Bruce > >>> > >>> On Fri, May 24, 2024 at 3:26 PM Martin Jansa > >>> wrote: > It was failing yesterday, but today it built OK, I didn't do any > runtime testing (only checked buildhistory-diff). > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > still failing for me. > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > wrote: > > Did you get this to build with just that change ? > > > > My patches to cni are significantly larger to get it to build and > > create binaries. > > > > Bruce > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > lists.yoctoproject.org > > wrote: > >> * see: > >> > >> https://lists.openembedded.org/g/openembedded-architecture/message/2007 > >> > >> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > >> > >> * fixes: > >>http://errors.yoctoproject.org/Errors/Details/772691/ > >>ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > >> TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > >> [license-checksum] > >> > >> Signed-off-by: Martin Jansa > >> --- > >> recipes-networking/cni/cni_git.bb | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/recipes-networking/cni/cni_git.bb > >> b/recipes-networking/cni/cni_git.bb > >> index b41b47cf..c0f316ec 100644 > >> --- a/recipes-networking/cni/cni_git.bb > >> +++ b/recipes-networking/cni/cni_git.bb > >> @@ -14,7 +14,7 @@ SRCREV_plugins = > >> "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > >> SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > >> SRCREV_FORMAT = "cni_plugins" > >> SRC_URI = "\ > >> - > >> git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > >> \ > >> + > >> git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > >> \ > >> > >> git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > >> \ > >> >
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
Hi Bruce, I did some investigation this afternoon. A patch series has been sent, which mostly comes from 'grep + sed', and also includes a patch from Martin. I did build tests, the result looks good to me. I think the only remaining recipes that need to be dealt with are: yq and possibly runv. I'll need to do something else tomorrow. Hope you or someone else could check more. Hope this would help a little bit. Regards, Qi On 5/28/24 04:37, Bruce Ashfield via lists.yoctoproject.org wrote: As an update. My patience is wearing thin. The posted patches don't build for me, my changes don't build for me, if I hack things, the runtime is broken. I'm seeing different fetching and source management based on the number of git repositories in the SCR_URI, which means I'm running into everything from missing source to license checksum failures when I reorganize the code. I'm stubborn about this sort of thing, there's no sense in me just merging patches that don't build on my machines, and/or patches that I don't understand how they fix issues, since I'll never be able to maintain things afterwards. I am still working on this, but I'm no debugging oe-core code to try and understand what is happening and why the behaviour is changing, so everything is extremely hacked up and I'm not even near a RFT branch yet. Summary: it will be quite some time before I have meta-virt working again, since I feel like I've had a bag of rocks dumped right on top of me and changes to core that I don't have straight in my head yet. Bruce On Fri, May 24, 2024 at 3:40 PM Martin Jansa wrote: On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield wrote: Interesting. I've been through the layer now and have done all the simple semantic changes to adjust things. I'm almost to runtime testing, but expect that by Monday I'll have everything sorted. OK, looking forward to the master-next update (and kirkstone update) to have a bit greener world builds again :). Bruce On Fri, May 24, 2024 at 3:26 PM Martin Jansa wrote: It was failing yesterday, but today it built OK, I didn't do any runtime testing (only checked buildhistory-diff). Have similar change for nerdctl cri-tools, only yq and docker-moby are still failing for me. On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield wrote: Did you get this to build with just that change ? My patches to cni are significantly larger to get it to build and create binaries. Bruce On Fri, May 24, 2024 at 3:09 PM Martin Jansa via lists.yoctoproject.org wrote: * see: https://lists.openembedded.org/g/openembedded-architecture/message/2007 https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 * fixes: http://errors.yoctoproject.org/Errors/Details/772691/ ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE [license-checksum] Signed-off-by: Martin Jansa --- recipes-networking/cni/cni_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/recipes-networking/cni/cni_git.bb b/recipes-networking/cni/cni_git.bb index b41b47cf..c0f316ec 100644 --- a/recipes-networking/cni/cni_git.bb +++ b/recipes-networking/cni/cni_git.bb @@ -14,7 +14,7 @@ SRCREV_plugins = "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" SRCREV_FORMAT = "cni_plugins" SRC_URI = "\ - git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https \ + git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} \ git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https \ git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel \ file://modules.txt \ -- 2.45.1 -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8778): https://lists.yoctoproject.org/g/meta-virtualization/message/8778 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
On Mon, May 27, 2024 at 4:38 PM Bruce Ashfield via lists.yoctoproject.org wrote: > > As an update. My patience is wearing thin. > > The posted patches don't build for me, my changes don't build for me, > if I hack things, the runtime is broken. > > I'm seeing different fetching and source management based on the > number of git repositories in the SCR_URI, which means I'm running > into everything from missing source to license checksum failures when > I reorganize the code. > > I'm stubborn about this sort of thing, there's no sense in me just > merging patches that don't build on my machines, and/or patches that I > don't understand how they fix issues, since I'll never be able to > maintain things afterwards. > > I am still working on this, but I'm no debugging oe-core code to try > and understand what is happening and why the behaviour is changing, so > everything is extremely hacked up and I'm not even near a RFT branch > yet. > > Summary: it will be quite some time before I have meta-virt working > again, since I feel like I've had a bag of rocks dumped right on top > of me and changes to core that I don't have straight in my head yet. I did finally come to a better understanding of what is breaking some of the packages and I've now fixed them. I'm working on some of the more complex container ones tomorrow, but I expect to have the branch ready for more testing on Tuesday. Bruce > > Bruce > > On Fri, May 24, 2024 at 3:40 PM Martin Jansa wrote: > > > > On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield > > wrote: > > > > > > Interesting. > > > > > > I've been through the layer now and have done all the simple semantic > > > changes to adjust things. > > > > > > I'm almost to runtime testing, but expect that by Monday I'll have > > > everything sorted. > > > > OK, looking forward to the master-next update (and kirkstone update) > > to have a bit greener world builds again :). > > > > > > > > Bruce > > > > > > On Fri, May 24, 2024 at 3:26 PM Martin Jansa > > > wrote: > > > > > > > > It was failing yesterday, but today it built OK, I didn't do any > > > > runtime testing (only checked buildhistory-diff). > > > > > > > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > > > > still failing for me. > > > > > > > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > > > > wrote: > > > > > > > > > > Did you get this to build with just that change ? > > > > > > > > > > My patches to cni are significantly larger to get it to build and > > > > > create binaries. > > > > > > > > > > Bruce > > > > > > > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > > > > lists.yoctoproject.org > > > > > wrote: > > > > > > > > > > > > * see: > > > > > > > > > > > > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > > > > > > > > > > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > > > > > > > > > > > * fixes: > > > > > > http://errors.yoctoproject.org/Errors/Details/772691/ > > > > > > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > > > > > > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > > > > > > [license-checksum] > > > > > > > > > > > > Signed-off-by: Martin Jansa > > > > > > --- > > > > > > recipes-networking/cni/cni_git.bb | 2 +- > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > diff --git a/recipes-networking/cni/cni_git.bb > > > > > > b/recipes-networking/cni/cni_git.bb > > > > > > index b41b47cf..c0f316ec 100644 > > > > > > --- a/recipes-networking/cni/cni_git.bb > > > > > > +++ b/recipes-networking/cni/cni_git.bb > > > > > > @@ -14,7 +14,7 @@ SRCREV_plugins = > > > > > > "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > > > > > > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > > > > > > SRCREV_FORMAT = "cni_plugins" > > > > > > SRC_URI = "\ > > > > > > - > > > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > > > > > > \ > > > > > > + > > > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > > > > > > \ > > > > > > > > > > > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > > > > > > \ > > > > > > > > > > > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > > > > > > \ > > > > > > file://modules.txt \ > > > > > > -- > > > > > > 2.45.1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > > > > thee at
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
As an update. My patience is wearing thin. The posted patches don't build for me, my changes don't build for me, if I hack things, the runtime is broken. I'm seeing different fetching and source management based on the number of git repositories in the SCR_URI, which means I'm running into everything from missing source to license checksum failures when I reorganize the code. I'm stubborn about this sort of thing, there's no sense in me just merging patches that don't build on my machines, and/or patches that I don't understand how they fix issues, since I'll never be able to maintain things afterwards. I am still working on this, but I'm no debugging oe-core code to try and understand what is happening and why the behaviour is changing, so everything is extremely hacked up and I'm not even near a RFT branch yet. Summary: it will be quite some time before I have meta-virt working again, since I feel like I've had a bag of rocks dumped right on top of me and changes to core that I don't have straight in my head yet. Bruce On Fri, May 24, 2024 at 3:40 PM Martin Jansa wrote: > > On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield > wrote: > > > > Interesting. > > > > I've been through the layer now and have done all the simple semantic > > changes to adjust things. > > > > I'm almost to runtime testing, but expect that by Monday I'll have > > everything sorted. > > OK, looking forward to the master-next update (and kirkstone update) > to have a bit greener world builds again :). > > > > > Bruce > > > > On Fri, May 24, 2024 at 3:26 PM Martin Jansa wrote: > > > > > > It was failing yesterday, but today it built OK, I didn't do any > > > runtime testing (only checked buildhistory-diff). > > > > > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > > > still failing for me. > > > > > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > > > wrote: > > > > > > > > Did you get this to build with just that change ? > > > > > > > > My patches to cni are significantly larger to get it to build and > > > > create binaries. > > > > > > > > Bruce > > > > > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > > > lists.yoctoproject.org > > > > wrote: > > > > > > > > > > * see: > > > > > > > > > > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > > > > > > > > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > > > > > > > > > * fixes: > > > > > http://errors.yoctoproject.org/Errors/Details/772691/ > > > > > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > > > > > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > > > > > [license-checksum] > > > > > > > > > > Signed-off-by: Martin Jansa > > > > > --- > > > > > recipes-networking/cni/cni_git.bb | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/recipes-networking/cni/cni_git.bb > > > > > b/recipes-networking/cni/cni_git.bb > > > > > index b41b47cf..c0f316ec 100644 > > > > > --- a/recipes-networking/cni/cni_git.bb > > > > > +++ b/recipes-networking/cni/cni_git.bb > > > > > @@ -14,7 +14,7 @@ SRCREV_plugins = > > > > > "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > > > > > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > > > > > SRCREV_FORMAT = "cni_plugins" > > > > > SRC_URI = "\ > > > > > - > > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > > > > > \ > > > > > + > > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > > > > > \ > > > > > > > > > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > > > > > \ > > > > > > > > > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > > > > > \ > > > > > file://modules.txt \ > > > > > -- > > > > > 2.45.1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > > > thee at its end > > > > - "Use the force Harry" - Gandalf, Star Trek II > > > > > > > > -- > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > thee at its end > > - "Use the force Harry" - Gandalf, Star Trek II -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8769): https://lists.yoctoproject.org/g/meta-virtualization/message/8769 Mute This Topic:
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
On Fri, May 24, 2024 at 9:32 PM Bruce Ashfield wrote: > > Interesting. > > I've been through the layer now and have done all the simple semantic > changes to adjust things. > > I'm almost to runtime testing, but expect that by Monday I'll have > everything sorted. OK, looking forward to the master-next update (and kirkstone update) to have a bit greener world builds again :). > > Bruce > > On Fri, May 24, 2024 at 3:26 PM Martin Jansa wrote: > > > > It was failing yesterday, but today it built OK, I didn't do any > > runtime testing (only checked buildhistory-diff). > > > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > > still failing for me. > > > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > > wrote: > > > > > > Did you get this to build with just that change ? > > > > > > My patches to cni are significantly larger to get it to build and > > > create binaries. > > > > > > Bruce > > > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > > lists.yoctoproject.org > > > wrote: > > > > > > > > * see: > > > > > > > > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > > > > > > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > > > > > > > * fixes: > > > > http://errors.yoctoproject.org/Errors/Details/772691/ > > > > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > > > > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > > > > [license-checksum] > > > > > > > > Signed-off-by: Martin Jansa > > > > --- > > > > recipes-networking/cni/cni_git.bb | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/recipes-networking/cni/cni_git.bb > > > > b/recipes-networking/cni/cni_git.bb > > > > index b41b47cf..c0f316ec 100644 > > > > --- a/recipes-networking/cni/cni_git.bb > > > > +++ b/recipes-networking/cni/cni_git.bb > > > > @@ -14,7 +14,7 @@ SRCREV_plugins = > > > > "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > > > > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > > > > SRCREV_FORMAT = "cni_plugins" > > > > SRC_URI = "\ > > > > - > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > > > > \ > > > > + > > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > > > > \ > > > > > > > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > > > > \ > > > > > > > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > > > > \ > > > > file://modules.txt \ > > > > -- > > > > 2.45.1 > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > > thee at its end > > > - "Use the force Harry" - Gandalf, Star Trek II > > > > -- > - Thou shalt not follow the NULL pointer, for chaos and madness await > thee at its end > - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8765): https://lists.yoctoproject.org/g/meta-virtualization/message/8765 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
Interesting. I've been through the layer now and have done all the simple semantic changes to adjust things. I'm almost to runtime testing, but expect that by Monday I'll have everything sorted. Bruce On Fri, May 24, 2024 at 3:26 PM Martin Jansa wrote: > > It was failing yesterday, but today it built OK, I didn't do any > runtime testing (only checked buildhistory-diff). > > Have similar change for nerdctl cri-tools, only yq and docker-moby are > still failing for me. > > On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield > wrote: > > > > Did you get this to build with just that change ? > > > > My patches to cni are significantly larger to get it to build and > > create binaries. > > > > Bruce > > > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > > lists.yoctoproject.org > > wrote: > > > > > > * see: > > > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > > > > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > > > > > * fixes: > > > http://errors.yoctoproject.org/Errors/Details/772691/ > > > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > > > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > > > [license-checksum] > > > > > > Signed-off-by: Martin Jansa > > > --- > > > recipes-networking/cni/cni_git.bb | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/recipes-networking/cni/cni_git.bb > > > b/recipes-networking/cni/cni_git.bb > > > index b41b47cf..c0f316ec 100644 > > > --- a/recipes-networking/cni/cni_git.bb > > > +++ b/recipes-networking/cni/cni_git.bb > > > @@ -14,7 +14,7 @@ SRCREV_plugins = > > > "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > > > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > > > SRCREV_FORMAT = "cni_plugins" > > > SRC_URI = "\ > > > - > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > > > \ > > > + > > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > > > \ > > > > > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > > > \ > > > > > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > > > \ > > > file://modules.txt \ > > > -- > > > 2.45.1 > > > > > > > > > > > > > > > > > > -- > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > thee at its end > > - "Use the force Harry" - Gandalf, Star Trek II -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8764): https://lists.yoctoproject.org/g/meta-virtualization/message/8764 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
It was failing yesterday, but today it built OK, I didn't do any runtime testing (only checked buildhistory-diff). Have similar change for nerdctl cri-tools, only yq and docker-moby are still failing for me. On Fri, May 24, 2024 at 9:22 PM Bruce Ashfield wrote: > > Did you get this to build with just that change ? > > My patches to cni are significantly larger to get it to build and > create binaries. > > Bruce > > On Fri, May 24, 2024 at 3:09 PM Martin Jansa via > lists.yoctoproject.org > wrote: > > > > * see: > > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > > > * fixes: > > http://errors.yoctoproject.org/Errors/Details/772691/ > > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > > [license-checksum] > > > > Signed-off-by: Martin Jansa > > --- > > recipes-networking/cni/cni_git.bb | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/recipes-networking/cni/cni_git.bb > > b/recipes-networking/cni/cni_git.bb > > index b41b47cf..c0f316ec 100644 > > --- a/recipes-networking/cni/cni_git.bb > > +++ b/recipes-networking/cni/cni_git.bb > > @@ -14,7 +14,7 @@ SRCREV_plugins = > > "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > > SRCREV_FORMAT = "cni_plugins" > > SRC_URI = "\ > > - > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > > \ > > + > > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > > \ > > > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > > \ > > > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > > \ > > file://modules.txt \ > > -- > > 2.45.1 > > > > > > > > > > > -- > - Thou shalt not follow the NULL pointer, for chaos and madness await > thee at its end > - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8761): https://lists.yoctoproject.org/g/meta-virtualization/message/8761 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
Did you get this to build with just that change ? My patches to cni are significantly larger to get it to build and create binaries. Bruce On Fri, May 24, 2024 at 3:09 PM Martin Jansa via lists.yoctoproject.org wrote: > > * see: > https://lists.openembedded.org/g/openembedded-architecture/message/2007 > > https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 > > * fixes: > http://errors.yoctoproject.org/Errors/Details/772691/ > ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: > TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE > [license-checksum] > > Signed-off-by: Martin Jansa > --- > recipes-networking/cni/cni_git.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/recipes-networking/cni/cni_git.bb > b/recipes-networking/cni/cni_git.bb > index b41b47cf..c0f316ec 100644 > --- a/recipes-networking/cni/cni_git.bb > +++ b/recipes-networking/cni/cni_git.bb > @@ -14,7 +14,7 @@ SRCREV_plugins = "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" > SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" > SRCREV_FORMAT = "cni_plugins" > SRC_URI = "\ > - > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https > \ > + > git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} > \ > > git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https > \ > > git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel > \ > file://modules.txt \ > -- > 2.45.1 > > > > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8759): https://lists.yoctoproject.org/g/meta-virtualization/message/8759 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[meta-virtualization] [RFC][PATCH 2/3] cni: use GO_SRCURI_DESTSUFFIX
* see: https://lists.openembedded.org/g/openembedded-architecture/message/2007 https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327 * fixes: http://errors.yoctoproject.org/Errors/Details/772691/ ERROR: QA Issue: cni: LIC_FILES_CHKSUM points to an invalid file: TOPDIR/tmp-glibc/work/core2-64-oe-linux/cni/v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/cni-v1.2.0-rc0+gitb62753aa2bfa365c1ceaff6f25774a8047c896b5/src/import/LICENSE [license-checksum] Signed-off-by: Martin Jansa --- recipes-networking/cni/cni_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/recipes-networking/cni/cni_git.bb b/recipes-networking/cni/cni_git.bb index b41b47cf..c0f316ec 100644 --- a/recipes-networking/cni/cni_git.bb +++ b/recipes-networking/cni/cni_git.bb @@ -14,7 +14,7 @@ SRCREV_plugins = "b6a0e0bc96906f0d3bd6bfcaab0b5ae72292f46c" SRCREV_flannel_plugin = "6464faacf5c00e25321573225d74638455ef03a0" SRCREV_FORMAT = "cni_plugins" SRC_URI = "\ - git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https \ + git://github.com/containernetworking/cni.git;branch=main;name=cni;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX} \ git://github.com/containernetworking/plugins.git;branch=main;destsuffix=${S}/src/github.com/containernetworking/plugins;name=plugins;protocol=https \ git://github.com/flannel-io/cni-plugin;branch=main;name=flannel_plugin;protocol=https;destsuffix=${S}/src/github.com/containernetworking/plugins/plugins/meta/flannel \ file://modules.txt \ -- 2.45.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#8756): https://lists.yoctoproject.org/g/meta-virtualization/message/8756 Mute This Topic: https://lists.yoctoproject.org/mt/106287452/21656 Group Owner: meta-virtualization+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-