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

Bump Golang to 1.17 in GHA #820

Merged
merged 1 commit into from
Sep 18, 2021
Merged

Bump Golang to 1.17 in GHA #820

merged 1 commit into from
Sep 18, 2021

Conversation

gliptak
Copy link
Contributor

@gliptak gliptak commented Sep 17, 2021

Signed-off-by: Gábor Lipták gliptak@gmail.com

Signed-off-by: Gábor Lipták <gliptak@gmail.com>
@ctz ctz merged commit af9f42e into rustls:main Sep 18, 2021
@djc
Copy link
Member

djc commented Sep 18, 2021

@ctz I'm confused: this looks like it broke bogo in CI. Do you plan to mitigate this?

@ctz
Copy link
Member

ctz commented Sep 18, 2021

Er, yes, I misread the status of this PR and merged it prematurely. Currently looking at it.

@gliptak
Copy link
Contributor Author

gliptak commented Sep 18, 2021

@ctz how https://github.com/rustls/boringssl (?) pulled into this build tree?

@ctz
Copy link
Member

ctz commented Sep 18, 2021

It happens in this script: https://github.com/rustls/rustls/blob/main/bogo/fetch-and-build

Hm, that sparse checkout business is missing out go.sum; that sounds like it might be a problem.

@gliptak gliptak mentioned this pull request Sep 18, 2021
@gliptak gliptak deleted the patch-2 branch September 19, 2021 15:27
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.

3 participants