Skip to content

Update to Go 1.22#2417

Merged
rhatdan merged 3 commits intocontainers:mainfrom
mtrmac:go1.22
Sep 4, 2024
Merged

Update to Go 1.22#2417
rhatdan merged 3 commits intocontainers:mainfrom
mtrmac:go1.22

Conversation

@mtrmac
Copy link
Copy Markdown
Contributor

@mtrmac mtrmac commented Sep 3, 2024

Cc: @Luap99

@lsm5 PTAL — marking as draft to let you review, to make sure this is not a problem for downstream.

@Luap99
Copy link
Copy Markdown
Member

Luap99 commented Sep 3, 2024

You will need something like containers/podman@54b8c4f to fix the packit config to no longer build in envs with go 1.21

@packit-as-a-service
Copy link
Copy Markdown

Ephemeral COPR build failed. @containers/packit-build please check.

Based on containers/podman@54b8c4f
by Paul Holzinger <pholzing@redhat.com> .

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Copy link
Copy Markdown
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Not a problem for downstream. We'll simply end up not updating f39 which will be going EOL in maybe a couple of months.

@mtrmac mtrmac marked this pull request as ready for review September 4, 2024 14:53
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Sep 4, 2024

Thanks! Ready for merging, then.

@rhatdan rhatdan merged commit 4c9ec00 into containers:main Sep 4, 2024
@mtrmac mtrmac deleted the go1.22 branch September 4, 2024 16:46
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants