Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make compatible with Go 1.21 (update quic-go) #815

Open
eclipseo opened this issue Aug 12, 2023 · 2 comments
Open

Make compatible with Go 1.21 (update quic-go) #815

eclipseo opened this issue Aug 12, 2023 · 2 comments
Assignees

Comments

@eclipseo
Copy link

Go 1.21 is around the corner and we already have the rc3 in Fedora Rawhide. It would be nice to update quic-go to at least v0.37.4 https://github.com/quic-go/quic-go/releases/tag/v0.37.4

Thank you.

@eclipseo eclipseo changed the title Make compatible aith Go 1.21 (update quic-go) Make compatible with Go 1.21 (update quic-go) Aug 12, 2023
eclipseo added a commit to eclipseo/receptor that referenced this issue Aug 12, 2023
This is necessary to run on Go 1.21.

Fix ansible#815.
@AaronH88 AaronH88 self-assigned this Aug 14, 2023
@AaronH88
Copy link
Contributor

Thanks for this issue and PR @eclipseo .
The quic-go 0.37 release not only supports Go 1.21 but it removed backward compatibility for any Go version less than 1.20.
As a result, this update will need to be carefully managed so as not to break any application build upon receptor.

Once I have confirmation that dropping support for Go 1.19 is ok I will merge.

Thanks once again.
CC @TheRealHaoLiu @shanemcd @fosterseth

@AaronH88
Copy link
Contributor

I just confirmed that receptor build uses go from RHEL and the latest released version is 1.19 right now. looks like 1.20 will be released in RHEL 8.9/9.3 mid-November. We will not be able to drop support for GO 1.19 until then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants