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

Add GO111MODULE=off to build.sh, use Golang 1.16 #966

Merged

Conversation

@timvaillancourt
Copy link
Contributor

@timvaillancourt timvaillancourt commented May 3, 2021

Related issue: #933

Description

This PR adds GO111MODULE=off to the go build line of various build scripts/configs so that builds succeed on Golang 1.16

This is a temporary solution until #935 is merged, in which case the GO111MODULE override can be removed. Currently this PR is blocked by some testing failures

Resolves #933

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.
@timvaillancourt timvaillancourt requested review from rashiq and gtowey May 3, 2021
@timvaillancourt timvaillancourt changed the title Add GO111MODULE=off to build.sh Add GO111MODULE=off to build.sh, use Golang 1.16 May 3, 2021
@timvaillancourt timvaillancourt added this to the v1.2.0 milestone May 5, 2021
@timvaillancourt timvaillancourt removed this from the v1.2.0 milestone May 5, 2021
@timvaillancourt timvaillancourt added this to the v1.1.1 milestone May 5, 2021
@timvaillancourt timvaillancourt removed this from the v1.1.1 milestone May 5, 2021
@timvaillancourt timvaillancourt added this to the v1.1.2 milestone May 5, 2021
gtowey
gtowey approved these changes May 24, 2021
@timvaillancourt timvaillancourt merged commit 8f42ded into github:master May 25, 2021
5 checks passed
@timvaillancourt timvaillancourt deleted the go-build-GO111MODULE-off branch May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants