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_context: set GOTOOLCHAIN to 'local' #3660

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

sluongng
Copy link
Contributor

Enforce usage of the current toolchain and avoid downloading new
toolchains during build.

What type of PR is this?

Uncomment one line below and remove others.

Bug fix
Feature
Documentation
Other

What does this PR do? Why is it needed?

Which issues(s) does this PR fix?

Fixes #

Other notes for review

go/private/context.bzl Outdated Show resolved Hide resolved
@sluongng sluongng force-pushed the sluongng/go-toolchain-local branch from afdd765 to b786e7b Compare August 15, 2023 14:18
Enforce usage of the current toolchain and avoid downloading new
toolchains during build.
@fmeum fmeum force-pushed the sluongng/go-toolchain-local branch from b786e7b to e0dac32 Compare August 15, 2023 14:20
@fmeum fmeum enabled auto-merge (squash) August 15, 2023 14:20
@fmeum fmeum merged commit 0da92cd into bazel-contrib:master Aug 15, 2023
sluongng added a commit to buildbuddy-io/buildbuddy that referenced this pull request Aug 28, 2023
Upgrade rules_go to latest commit in `master` at the moment of writing.

Unblocks Go 1.21 upgrade in #4516 by including the fixes in:
- bazel-contrib/rules_go#3660
- bazel-contrib/rules_go#3666
sluongng added a commit to buildbuddy-io/buildbuddy that referenced this pull request Aug 29, 2023
Upgrade rules_go to latest commit in `master` at the moment of writing.

Unblocks Go 1.21 upgrade in #4516 by including the fixes in:
- bazel-contrib/rules_go#3660
- bazel-contrib/rules_go#3666
fmeum pushed a commit to bazel-contrib/bazel-gazelle that referenced this pull request Aug 12, 2024
**What type of PR is this?**

Bug fix

**What package or component does this PR mostly affect?**

go_repository_cache

**What does this PR do? Why is it needed?**

Override the **GOTOOLCHAIN** local environment parameter defined by `go
env -w` to use the local Go environment made available by `gazelle`.

**Which issues(s) does this PR fix?**

Fixes #1858

**Other notes for review**

Same fix than bazel-contrib/rules_go#3660 in
[rules_go](https://github.com/bazelbuild/rules_go).
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