Re: [meta-virtualization] breakages in go recipes

2024-05-23 Thread Bruce Ashfield
Hi all,

I'm back from my travels and have already fixed a bunch of the issues.

I'll push my WIP changes to master-next by end of day Friday, and then
I suggest we jump off those changes.

The ones on master-next are based on what I was doing before I headed
out for vacation.

I'm mainly sorting out some remaining issues on the more complex
recipes (cni, etc) and I'll be good to share the progress.

Bruce

On Thu, May 23, 2024 at 4:10 PM Martin Jansa  wrote:
>
> Hi Theodore,
>
> yes, I'm fighting similar issues in our layers today as well.
>
> I've integrated your change and checked that runc builds now.
>
> Here are the public logs for all the meta-virtualzation issues
> http://errors.yoctoproject.org/Errors/Build/184253/
> I'll update the WIP changes with these and send what I have after
> checking at least buildhistory-diff.
>
>
> On Thu, May 23, 2024 at 8:23 PM Theodore A. Roth  wrote:
> >
> >
> > On Thu, May 23, 2024 at 2:34 AM Martin Jansa  wrote:
> >>
> >> FWIW: My world build with oe-core
> >> master:094273bd7d1768e14fbdcd2f239bee14c630a625 (so before the main
> >> UNPACKDIR changes) finished with these failures from
> >> meta-virtualization:
> >>
> >> ERROR: Task 
> >> (meta-virtualization/recipes-containers/cri-tools/cri-tools_git.bb:do_patch)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-containers/docker-compose/docker-compose_git.bb:do_populate_lic)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-containers/docker/docker-moby_git.bb:do_patch)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-containers/nerdctl/nerdctl_git.bb:do_patch)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-containers/runc/runc-opencontainers_git.bb:do_patch)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_compile)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_populate_lic)
> >> failed with exit code '1'
> >> ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_compile)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_populate_lic)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-networking/cni/cni_git.bb:do_compile)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-networking/cni/cni_git.bb:do_populate_lic)
> >> failed with exit code '1'
> >> ERROR: Task 
> >> (meta-virtualization/recipes-networking/netns/netns_git.bb:do_patch)
> >> failed with exit code '1'
> >>
> >> the UNPACKDIR changes will probably cause few more issues, I've
> >> already noticed one with linux-yocto_virtualization.inc and added
> >> temporary work around here:
> >> https://github.com/shr-project/meta-virtualization/commits/jansa/master/
> >>
> >> will send what I have later today as a RFC (as I cannot even build
> >> whole image yet) unless Theodore gets to it sooner.
> >
> >
> > Martin:
> >
> > Spent most of the day fighting with UNPACKDIR breakages outside of 
> > meta-virtualization.
> > Finally got to the point where I could build your WIP changes. Found some 
> > issues with the
> > runc recipes and have pushed fixes here:
> >
> > https://github.com/troth/meta-virtualization/commit/f604f1fb39fad9f496700be59333edcdc3e7a633
> >
> > I'm still waiting for a full build to complete, so I have not completely 
> > verified the rest of your changes.
> >
> > Ted Roth



-- 
- 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 (#8745): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8745
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-23 Thread Martin Jansa
Hi Theodore,

yes, I'm fighting similar issues in our layers today as well.

I've integrated your change and checked that runc builds now.

Here are the public logs for all the meta-virtualzation issues
http://errors.yoctoproject.org/Errors/Build/184253/
I'll update the WIP changes with these and send what I have after
checking at least buildhistory-diff.


On Thu, May 23, 2024 at 8:23 PM Theodore A. Roth  wrote:
>
>
> On Thu, May 23, 2024 at 2:34 AM Martin Jansa  wrote:
>>
>> FWIW: My world build with oe-core
>> master:094273bd7d1768e14fbdcd2f239bee14c630a625 (so before the main
>> UNPACKDIR changes) finished with these failures from
>> meta-virtualization:
>>
>> ERROR: Task 
>> (meta-virtualization/recipes-containers/cri-tools/cri-tools_git.bb:do_patch)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-containers/docker-compose/docker-compose_git.bb:do_populate_lic)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-containers/docker/docker-moby_git.bb:do_patch)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-containers/nerdctl/nerdctl_git.bb:do_patch)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-containers/runc/runc-opencontainers_git.bb:do_patch)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_compile)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_populate_lic)
>> failed with exit code '1'
>> ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_compile)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_populate_lic)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-networking/cni/cni_git.bb:do_compile)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-networking/cni/cni_git.bb:do_populate_lic)
>> failed with exit code '1'
>> ERROR: Task 
>> (meta-virtualization/recipes-networking/netns/netns_git.bb:do_patch)
>> failed with exit code '1'
>>
>> the UNPACKDIR changes will probably cause few more issues, I've
>> already noticed one with linux-yocto_virtualization.inc and added
>> temporary work around here:
>> https://github.com/shr-project/meta-virtualization/commits/jansa/master/
>>
>> will send what I have later today as a RFC (as I cannot even build
>> whole image yet) unless Theodore gets to it sooner.
>
>
> Martin:
>
> Spent most of the day fighting with UNPACKDIR breakages outside of 
> meta-virtualization.
> Finally got to the point where I could build your WIP changes. Found some 
> issues with the
> runc recipes and have pushed fixes here:
>
> https://github.com/troth/meta-virtualization/commit/f604f1fb39fad9f496700be59333edcdc3e7a633
>
> I'm still waiting for a full build to complete, so I have not completely 
> verified the rest of your changes.
>
> Ted Roth

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8744): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8744
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-23 Thread Theodore A. Roth
On Thu, May 23, 2024 at 2:34 AM Martin Jansa  wrote:

> FWIW: My world build with oe-core
> master:094273bd7d1768e14fbdcd2f239bee14c630a625 (so before the main
> UNPACKDIR changes) finished with these failures from
> meta-virtualization:
>
> ERROR: Task
> (meta-virtualization/recipes-containers/cri-tools/cri-tools_git.bb:
> do_patch)
> failed with exit code '1'
> ERROR: Task
> (meta-virtualization/recipes-containers/docker-compose/docker-compose_git.bb:
> do_populate_lic)
> failed with exit code '1'
> ERROR: Task
> (meta-virtualization/recipes-containers/docker/docker-moby_git.bb:
> do_patch)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-containers/nerdctl/nerdctl_git.bb:
> do_patch)
> failed with exit code '1'
> ERROR: Task
> (meta-virtualization/recipes-containers/runc/runc-opencontainers_git.bb:
> do_patch)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:
> do_compile)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-devtools/go/go-md2man_git.bb:
> do_populate_lic)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_compile)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:
> do_populate_lic)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-networking/cni/cni_git.bb:
> do_compile)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-networking/cni/cni_git.bb:
> do_populate_lic)
> failed with exit code '1'
> ERROR: Task (meta-virtualization/recipes-networking/netns/netns_git.bb
> :do_patch)
> failed with exit code '1'
>
> the UNPACKDIR changes will probably cause few more issues, I've
> already noticed one with linux-yocto_virtualization.inc and added
> temporary work around here:
> https://github.com/shr-project/meta-virtualization/commits/jansa/master/
>
> will send what I have later today as a RFC (as I cannot even build
> whole image yet) unless Theodore gets to it sooner.
>

Martin:

Spent most of the day fighting with UNPACKDIR breakages outside of
meta-virtualization.
Finally got to the point where I could build your WIP changes. Found some
issues with the
runc recipes and have pushed fixes here:

https://github.com/troth/meta-virtualization/commit/f604f1fb39fad9f496700be59333edcdc3e7a633

I'm still waiting for a full build to complete, so I have not completely
verified the rest of your changes.

Ted Roth

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8743): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8743
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-23 Thread Martin Jansa
FWIW: My world build with oe-core
master:094273bd7d1768e14fbdcd2f239bee14c630a625 (so before the main
UNPACKDIR changes) finished with these failures from
meta-virtualization:

ERROR: Task 
(meta-virtualization/recipes-containers/cri-tools/cri-tools_git.bb:do_patch)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-containers/docker-compose/docker-compose_git.bb:do_populate_lic)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-containers/docker/docker-moby_git.bb:do_patch)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-containers/nerdctl/nerdctl_git.bb:do_patch)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-containers/runc/runc-opencontainers_git.bb:do_patch)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_compile)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-devtools/go/go-md2man_git.bb:do_populate_lic)
failed with exit code '1'
ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_compile)
failed with exit code '1'
ERROR: Task (meta-virtualization/recipes-devtools/yq/yq_git.bb:do_populate_lic)
failed with exit code '1'
ERROR: Task (meta-virtualization/recipes-networking/cni/cni_git.bb:do_compile)
failed with exit code '1'
ERROR: Task 
(meta-virtualization/recipes-networking/cni/cni_git.bb:do_populate_lic)
failed with exit code '1'
ERROR: Task (meta-virtualization/recipes-networking/netns/netns_git.bb:do_patch)
failed with exit code '1'

the UNPACKDIR changes will probably cause few more issues, I've
already noticed one with linux-yocto_virtualization.inc and added
temporary work around here:
https://github.com/shr-project/meta-virtualization/commits/jansa/master/

will send what I have later today as a RFC (as I cannot even build
whole image yet) unless Theodore gets to it sooner.

On Thu, May 23, 2024 at 6:04 AM Bruce Ashfield  wrote:
>
> Martin's advice is on point (as usual).
>
> I'll be travelling all day tomorrow, so I won't even have time to
> review or merge until Friday.
>
> I was aware of the pending changes, but I've been on the road for a
> bit now, and didn't have a chance to complete my changes.
>
> Bruce
>
> On Wed, May 22, 2024 at 6:54 PM Martin Jansa via
> lists.yoctoproject.org 
> wrote:
> >
> > I think one per recipe (or few related ones like runc*bb) would be
> > easier to review in this case.
> >
> > On Thu, May 23, 2024 at 12:51 AM Theodore A. Roth  wrote:
> > >
> > >
> > > On Wed, May 22, 2024 at 4:05 PM Martin Jansa  
> > > wrote:
> > >>
> > >> Hi,
> > >>
> > >> I can confirm that I'm seeing the same build failures since this
> > >> morning and agree that the recipes need to adapt to:
> > >> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327
> > >>
> > >> I've started with some WIP changes and fixed some of them in:
> > >> https://github.com/shr-project/meta-virtualization/commit/30a85c0319c253632206b0d2c2c483f11a490939
> > >> but than got side tracked by some other build failures and not sure
> > >> when I'll be able to get back to it.
> > >>
> > >> I've also asked Bruce on IRC and he is aware of the issues, but might
> > >> not have time to resolve it soon, so if you have the changes ready,
> > >> then please send them to ML for Bruce to review when he has time and
> > >> I'll happily build test them as well.
> > >>
> > >> Cheers,
> > >
> > >
> > > Should I send a single patch fixing a bunch of recipes (aka many 
> > > destsuffix fixes at once) or one patch per recipe?
> > >
> > > Ted Roth
> > >
> > >>
> > >>
> > >> On Wed, May 22, 2024 at 11:52 PM Theodore A. Roth via
> > >> lists.yoctoproject.org 
> > >> wrote:
> > >> >
> > >> > Hi,
> > >> >
> > >> > In our CI builds for multiple projects using meta-virtualization 
> > >> > master branch, we are
> > >> > seeing some build failures for some recipes that use GO:
> > >> >
> > >> > * go-md2man
> > >> > * runc-opencontainers
> > >> > * docker-moby
> > >> > * podman
> > >> > * buildah
> > >> > * cni
> > >> >
> > >> > I think I've narrowed the breakages down to this commit to oe-core 
> > >> > from Richard Purdie:
> > >> >
> > >> > * https://lists.openembedded.org/g/openembedded-core/message/199614
> > >> >
> > >> > The fixes for our builds seem simple enough. Just add 
> > >> > ';destsuffix=${GO_SRCURI_DESTSUFFIX}'
> > >> > to the SRC_URI. For example:
> > >> >
> > >> > ---
> > >> >
> > >> > diff --git a/recipes-devtools/go/go-md2man_git.bb 
> > >> > b/recipes-devtools/go/go-md2man_git.bb
> > >> > index 1d96bb1b..73d8d167 100644
> > >> > --- a/recipes-devtools/go/go-md2man_git.bb
> > >> > +++ b/recipes-devtools/go/go-md2man_git.bb
> > >> > @@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
> > >> >  GO_IMPORT = "github.com/cpuguy83/go-md2man"
> > >> >  #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"
> > >> >
> > >> > -SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
> > >> > +SRC_URI = 
> > >> > 

Re: [meta-virtualization] breakages in go recipes

2024-05-22 Thread Bruce Ashfield
Martin's advice is on point (as usual).

I'll be travelling all day tomorrow, so I won't even have time to
review or merge until Friday.

I was aware of the pending changes, but I've been on the road for a
bit now, and didn't have a chance to complete my changes.

Bruce

On Wed, May 22, 2024 at 6:54 PM Martin Jansa via
lists.yoctoproject.org 
wrote:
>
> I think one per recipe (or few related ones like runc*bb) would be
> easier to review in this case.
>
> On Thu, May 23, 2024 at 12:51 AM Theodore A. Roth  wrote:
> >
> >
> > On Wed, May 22, 2024 at 4:05 PM Martin Jansa  wrote:
> >>
> >> Hi,
> >>
> >> I can confirm that I'm seeing the same build failures since this
> >> morning and agree that the recipes need to adapt to:
> >> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327
> >>
> >> I've started with some WIP changes and fixed some of them in:
> >> https://github.com/shr-project/meta-virtualization/commit/30a85c0319c253632206b0d2c2c483f11a490939
> >> but than got side tracked by some other build failures and not sure
> >> when I'll be able to get back to it.
> >>
> >> I've also asked Bruce on IRC and he is aware of the issues, but might
> >> not have time to resolve it soon, so if you have the changes ready,
> >> then please send them to ML for Bruce to review when he has time and
> >> I'll happily build test them as well.
> >>
> >> Cheers,
> >
> >
> > Should I send a single patch fixing a bunch of recipes (aka many destsuffix 
> > fixes at once) or one patch per recipe?
> >
> > Ted Roth
> >
> >>
> >>
> >> On Wed, May 22, 2024 at 11:52 PM Theodore A. Roth via
> >> lists.yoctoproject.org 
> >> wrote:
> >> >
> >> > Hi,
> >> >
> >> > In our CI builds for multiple projects using meta-virtualization master 
> >> > branch, we are
> >> > seeing some build failures for some recipes that use GO:
> >> >
> >> > * go-md2man
> >> > * runc-opencontainers
> >> > * docker-moby
> >> > * podman
> >> > * buildah
> >> > * cni
> >> >
> >> > I think I've narrowed the breakages down to this commit to oe-core from 
> >> > Richard Purdie:
> >> >
> >> > * https://lists.openembedded.org/g/openembedded-core/message/199614
> >> >
> >> > The fixes for our builds seem simple enough. Just add 
> >> > ';destsuffix=${GO_SRCURI_DESTSUFFIX}'
> >> > to the SRC_URI. For example:
> >> >
> >> > ---
> >> >
> >> > diff --git a/recipes-devtools/go/go-md2man_git.bb 
> >> > b/recipes-devtools/go/go-md2man_git.bb
> >> > index 1d96bb1b..73d8d167 100644
> >> > --- a/recipes-devtools/go/go-md2man_git.bb
> >> > +++ b/recipes-devtools/go/go-md2man_git.bb
> >> > @@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
> >> >  GO_IMPORT = "github.com/cpuguy83/go-md2man"
> >> >  #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"
> >> >
> >> > -SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
> >> > +SRC_URI = 
> >> > "git://${GO_IMPORT}.git;branch=master;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX}"
> >> >
> >> >  SRCREV = "f79a8a8ca69da163eee19ab442bedad7a35bba5a"
> >> >  PV = "1.0.10+git"
> >> >
> >> > ---
> >> >
> >> > I plan to finish my testing for our breakages and submit some patches, 
> >> > but it feels like there are many
> >> > more recipes that will need this change made (basically anything that is 
> >> > using 'inherit go').
> >> >
> >> > I'm a bit reluctant to submit patches for recipes we are not using (and 
> >> > testing) even if they seem like
> >> > mechanical changes, but I wanted to let the list know what I have found.
> >> >
> >> > Ted Roth
> >> >
> >> >
> >> >
>
> 
>


-- 
- 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 (#8741): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8741
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-22 Thread Martin Jansa
I think one per recipe (or few related ones like runc*bb) would be
easier to review in this case.

On Thu, May 23, 2024 at 12:51 AM Theodore A. Roth  wrote:
>
>
> On Wed, May 22, 2024 at 4:05 PM Martin Jansa  wrote:
>>
>> Hi,
>>
>> I can confirm that I'm seeing the same build failures since this
>> morning and agree that the recipes need to adapt to:
>> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327
>>
>> I've started with some WIP changes and fixed some of them in:
>> https://github.com/shr-project/meta-virtualization/commit/30a85c0319c253632206b0d2c2c483f11a490939
>> but than got side tracked by some other build failures and not sure
>> when I'll be able to get back to it.
>>
>> I've also asked Bruce on IRC and he is aware of the issues, but might
>> not have time to resolve it soon, so if you have the changes ready,
>> then please send them to ML for Bruce to review when he has time and
>> I'll happily build test them as well.
>>
>> Cheers,
>
>
> Should I send a single patch fixing a bunch of recipes (aka many destsuffix 
> fixes at once) or one patch per recipe?
>
> Ted Roth
>
>>
>>
>> On Wed, May 22, 2024 at 11:52 PM Theodore A. Roth via
>> lists.yoctoproject.org 
>> wrote:
>> >
>> > Hi,
>> >
>> > In our CI builds for multiple projects using meta-virtualization master 
>> > branch, we are
>> > seeing some build failures for some recipes that use GO:
>> >
>> > * go-md2man
>> > * runc-opencontainers
>> > * docker-moby
>> > * podman
>> > * buildah
>> > * cni
>> >
>> > I think I've narrowed the breakages down to this commit to oe-core from 
>> > Richard Purdie:
>> >
>> > * https://lists.openembedded.org/g/openembedded-core/message/199614
>> >
>> > The fixes for our builds seem simple enough. Just add 
>> > ';destsuffix=${GO_SRCURI_DESTSUFFIX}'
>> > to the SRC_URI. For example:
>> >
>> > ---
>> >
>> > diff --git a/recipes-devtools/go/go-md2man_git.bb 
>> > b/recipes-devtools/go/go-md2man_git.bb
>> > index 1d96bb1b..73d8d167 100644
>> > --- a/recipes-devtools/go/go-md2man_git.bb
>> > +++ b/recipes-devtools/go/go-md2man_git.bb
>> > @@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
>> >  GO_IMPORT = "github.com/cpuguy83/go-md2man"
>> >  #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"
>> >
>> > -SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
>> > +SRC_URI = 
>> > "git://${GO_IMPORT}.git;branch=master;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX}"
>> >
>> >  SRCREV = "f79a8a8ca69da163eee19ab442bedad7a35bba5a"
>> >  PV = "1.0.10+git"
>> >
>> > ---
>> >
>> > I plan to finish my testing for our breakages and submit some patches, but 
>> > it feels like there are many
>> > more recipes that will need this change made (basically anything that is 
>> > using 'inherit go').
>> >
>> > I'm a bit reluctant to submit patches for recipes we are not using (and 
>> > testing) even if they seem like
>> > mechanical changes, but I wanted to let the list know what I have found.
>> >
>> > Ted Roth
>> >
>> > 
>> >

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8740): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8740
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-22 Thread Theodore A. Roth
On Wed, May 22, 2024 at 4:05 PM Martin Jansa  wrote:

> Hi,
>
> I can confirm that I'm seeing the same build failures since this
> morning and agree that the recipes need to adapt to:
>
> https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327
>
> I've started with some WIP changes and fixed some of them in:
>
> https://github.com/shr-project/meta-virtualization/commit/30a85c0319c253632206b0d2c2c483f11a490939
> but than got side tracked by some other build failures and not sure
> when I'll be able to get back to it.
>
> I've also asked Bruce on IRC and he is aware of the issues, but might
> not have time to resolve it soon, so if you have the changes ready,
> then please send them to ML for Bruce to review when he has time and
> I'll happily build test them as well.
>
> Cheers,
>

Should I send a single patch fixing a bunch of recipes (aka many destsuffix
fixes at once) or one patch per recipe?

Ted Roth


>
> On Wed, May 22, 2024 at 11:52 PM Theodore A. Roth via
> lists.yoctoproject.org 
> wrote:
> >
> > Hi,
> >
> > In our CI builds for multiple projects using meta-virtualization master
> branch, we are
> > seeing some build failures for some recipes that use GO:
> >
> > * go-md2man
> > * runc-opencontainers
> > * docker-moby
> > * podman
> > * buildah
> > * cni
> >
> > I think I've narrowed the breakages down to this commit to oe-core from
> Richard Purdie:
> >
> > * https://lists.openembedded.org/g/openembedded-core/message/199614
> >
> > The fixes for our builds seem simple enough. Just add
> ';destsuffix=${GO_SRCURI_DESTSUFFIX}'
> > to the SRC_URI. For example:
> >
> > ---
> >
> > diff --git a/recipes-devtools/go/go-md2man_git.bb b/recipes-devtools/go/
> go-md2man_git.bb
> > index 1d96bb1b..73d8d167 100644
> > --- a/recipes-devtools/go/go-md2man_git.bb
> > +++ b/recipes-devtools/go/go-md2man_git.bb
> > @@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
> >  GO_IMPORT = "github.com/cpuguy83/go-md2man"
> >  #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"
> >
> > -SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
> > +SRC_URI =
> "git://${GO_IMPORT}.git;branch=master;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX}"
> >
> >  SRCREV = "f79a8a8ca69da163eee19ab442bedad7a35bba5a"
> >  PV = "1.0.10+git"
> >
> > ---
> >
> > I plan to finish my testing for our breakages and submit some patches,
> but it feels like there are many
> > more recipes that will need this change made (basically anything that is
> using 'inherit go').
> >
> > I'm a bit reluctant to submit patches for recipes we are not using (and
> testing) even if they seem like
> > mechanical changes, but I wanted to let the list know what I have found.
> >
> > Ted Roth
> >
> > 
> >
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8739): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8739
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/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] breakages in go recipes

2024-05-22 Thread Martin Jansa
Hi,

I can confirm that I'm seeing the same build failures since this
morning and agree that the recipes need to adapt to:
https://git.openembedded.org/openembedded-core/commit/?id=cc4ec43a2b657fb4c58429ab14f1edc2473c1327

I've started with some WIP changes and fixed some of them in:
https://github.com/shr-project/meta-virtualization/commit/30a85c0319c253632206b0d2c2c483f11a490939
but than got side tracked by some other build failures and not sure
when I'll be able to get back to it.

I've also asked Bruce on IRC and he is aware of the issues, but might
not have time to resolve it soon, so if you have the changes ready,
then please send them to ML for Bruce to review when he has time and
I'll happily build test them as well.

Cheers,

On Wed, May 22, 2024 at 11:52 PM Theodore A. Roth via
lists.yoctoproject.org 
wrote:
>
> Hi,
>
> In our CI builds for multiple projects using meta-virtualization master 
> branch, we are
> seeing some build failures for some recipes that use GO:
>
> * go-md2man
> * runc-opencontainers
> * docker-moby
> * podman
> * buildah
> * cni
>
> I think I've narrowed the breakages down to this commit to oe-core from 
> Richard Purdie:
>
> * https://lists.openembedded.org/g/openembedded-core/message/199614
>
> The fixes for our builds seem simple enough. Just add 
> ';destsuffix=${GO_SRCURI_DESTSUFFIX}'
> to the SRC_URI. For example:
>
> ---
>
> diff --git a/recipes-devtools/go/go-md2man_git.bb 
> b/recipes-devtools/go/go-md2man_git.bb
> index 1d96bb1b..73d8d167 100644
> --- a/recipes-devtools/go/go-md2man_git.bb
> +++ b/recipes-devtools/go/go-md2man_git.bb
> @@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
>  GO_IMPORT = "github.com/cpuguy83/go-md2man"
>  #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"
>
> -SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
> +SRC_URI = 
> "git://${GO_IMPORT}.git;branch=master;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX}"
>
>  SRCREV = "f79a8a8ca69da163eee19ab442bedad7a35bba5a"
>  PV = "1.0.10+git"
>
> ---
>
> I plan to finish my testing for our breakages and submit some patches, but it 
> feels like there are many
> more recipes that will need this change made (basically anything that is 
> using 'inherit go').
>
> I'm a bit reluctant to submit patches for recipes we are not using (and 
> testing) even if they seem like
> mechanical changes, but I wanted to let the list know what I have found.
>
> Ted Roth
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8738): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8738
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/21656
Group Owner: meta-virtualization+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[meta-virtualization] breakages in go recipes

2024-05-22 Thread Theodore A. Roth
Hi,

In our CI builds for multiple projects using meta-virtualization master
branch, we are
seeing some build failures for some recipes that use GO:

* go-md2man
* runc-opencontainers
* docker-moby
* podman
* buildah
* cni

I think I've narrowed the breakages down to this commit to oe-core from
Richard Purdie:

* https://lists.openembedded.org/g/openembedded-core/message/199614

The fixes for our builds seem simple enough. Just add
';destsuffix=${GO_SRCURI_DESTSUFFIX}'
to the SRC_URI. For example:

---

diff --git a/recipes-devtools/go/go-md2man_git.bb b/recipes-devtools/go/
go-md2man_git.bb
index 1d96bb1b..73d8d167 100644
--- a/recipes-devtools/go/go-md2man_git.bb
+++ b/recipes-devtools/go/go-md2man_git.bb
@@ -9,7 +9,7 @@ BBCLASSEXTEND = "native"
 GO_IMPORT = "github.com/cpuguy83/go-md2man"
 #GO_INSTALL = "${GO_IMPORT}/bin/go-md2man"

-SRC_URI = "git://${GO_IMPORT}.git;branch=master;protocol=https"
+SRC_URI =
"git://${GO_IMPORT}.git;branch=master;protocol=https;destsuffix=${GO_SRCURI_DESTSUFFIX}"

 SRCREV = "f79a8a8ca69da163eee19ab442bedad7a35bba5a"
 PV = "1.0.10+git"

---

I plan to finish my testing for our breakages and submit some patches, but
it feels like there are many
more recipes that will need this change made (basically anything that is
using 'inherit go').

I'm a bit reluctant to submit patches for recipes we are not using (and
testing) even if they seem like
mechanical changes, but I wanted to let the list know what I have found.

Ted Roth

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8737): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8737
Mute This Topic: https://lists.yoctoproject.org/mt/106252100/21656
Group Owner: meta-virtualization+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-