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: add a go.mod #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

go.mod: add a go.mod #1

wants to merge 2 commits into from

Conversation

rschio
Copy link

@rschio rschio commented May 10, 2024

Add a go.mod to enable build and pin the dependencies.

I used the Go version 1.22.3 because it's the version I'm using but maybe we should change it...

Another thing, I've noticed the https://github.com/kurin/blazer says its is no longer actively maintained and you can find the currently maintained repository at https://github.com/backblaze/blazer, but I kept the old one because it's working.

Add a go.mod to enable build and pin the dependencies versions.
@n2vi
Copy link

n2vi commented May 13, 2024

I am not qualified to review a b2 PR, but note that go1.20 may be more portable in go.mod because of transition issues with the new toolchain downloads. On my system, these coredump, which I work around by setting toolchain=local.

But if you're actually using new generics features or whatever, sure, go for go1.22.3. That's the Go version I have installed on my machine as well.

@rschio
Copy link
Author

rschio commented May 13, 2024

I didn't change the code, so probably a 6 year old Go version would work. I will set Go 1.20 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 this pull request may close these issues.

2 participants