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

go.mod: restore Go 1.22 support #262

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Conversation

AkihiroSuda
Copy link
Contributor

Go 1.22 has not reached its EOL yet

deitch
deitch previously approved these changes Oct 5, 2024
Copy link
Collaborator

@deitch deitch left a comment

Choose a reason for hiding this comment

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

Thanks @AkihiroSuda

@deitch
Copy link
Collaborator

deitch commented Oct 5, 2024

There's some crypto changes that need to be changed for this to pass. See the failing tests.

@AkihiroSuda
Copy link
Contributor Author

https://github.com/diskfs/go-diskfs/actions/runs/11157473532/job/31122367815?pr=262

Error: vet: filesystem/fat32/fat32_test.go:267:20: chacha.Read undefined (type *rand.ChaCha8 has no field or method Read)

Go 1.22 has not reached its EOL yet.

`math/rand/v2.(*Chacha8).Read` is replaced with `crypto/rand.Read`,
as the former one is not available in Go 1.22.

Signed-off-by: Akihiro Suda <[email protected]>
@deitch
Copy link
Collaborator

deitch commented Oct 7, 2024

Yeah. See my comment on the other issue. Just switch it back to crypto/rand in this PR and it should be clean.

@deitch
Copy link
Collaborator

deitch commented Oct 7, 2024

All looks good. You ready for merge @AkihiroSuda ?

@AkihiroSuda
Copy link
Contributor Author

All looks good. You ready for merge @AkihiroSuda ?

Yes, thanks

@deitch deitch merged commit f66ea89 into diskfs:master Oct 7, 2024
20 checks passed
@deitch
Copy link
Collaborator

deitch commented Oct 7, 2024

Thanks for driving this @AkihiroSuda

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 this pull request may close these issues.

2 participants