-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
go 1.16 #71289
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 1.16 #71289
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I'd say we should bootstrap with a stable version. The only reason it was a beta before is because that was the only possible option. |
This comment has been minimized.
This comment has been minimized.
If the |
Not necessarily. However, if go1.15 satisfies the requirements for versioned formulae, it may be possible to add one. That said, we usually only maintain five versions of a formula at a time, and at the moment there are |
Two questions on this.
And thanks for the awesome work your doing maintaining this! |
It's part of the official install instructions.
If you can find a way for that to download a checksummed dependency so we can be sure of the version and origin of it I think that would be a good change to make. |
Consider using Edit: Fixed typo; this comment was meant to be about the new |
That doesn't pin the checksum though so it's still vulnerable to various security issues. |
Well that very much depends on how one interpret "Install additional tools". For me, "additional" means something optional. I would be fine if this Formula came without |
It is also optional for me. And also there is no godoc in the archive package, https://golang.org/dl/go1.16.darwin-amd64.tar.gz |
|
Generally, I tend to agree that godoc could be dropped from the formula. It's probably in there because it used to be shipped with Go releases, and we have docs on the installation page for that reason. Maybe the caveats can direct the user to |
On ARM: Error: 180 failed steps!
Most of these are probably formulae without ARM bottles. On Mojave: Error: 93 failed steps!
On Catalina: Error: 94 failed steps!
Big Sur: Error: 94 failed steps!
|
A lot of the Mojave failures are from formulae that put the repository in Those should be fixable by just not doing the |
@carlocab, Ok looks like there are only 4 more failures which i couldn't quickly figure out. Would love some help otherwise will give it a shot again tomorrow. @circa10a could we rebase and see what fails once again to make sure that the only failures are the ones in the list above? I wonder if we missed any. |
The Go project provides an option for this (that's also cross-platform). You can download any version, including beta and release candidate versions, that live separately from your existing Homebrew installation. For example:
More info available here: https://golang.org/doc/manage-install#installing-multiple Except in extreme circumstances (eg, Apple Silicon) Homebrew will never ship beta or release candidate versions, but those are the versions you should be testing your own packages with to shake out these kinds of issues before the stable version is released. This is a good way to do so without relying on Homebrew or messing up a stable installation. |
3760869
@roopakv sorry for the delay. Rebased |
There's a couple PRs not quite merged yet. I'll rebase again when they're merged. |
Co-authored-by: Bo Anderson <[email protected]>
I've now rebased again on master, which should have everything fixed now. I've made a couple changes based on discussions here and in other issues:
|
There are 99 failures on ARM, but only one of them has been bottled:
This looks like a random network failure, so we can probably ignore it. |
Just a couple failures, but I've already fixed them and will open a separate PR for them. I'll go ahead and merge this now. Just to clarify this point made:
This isn't quite true. Depending on What we don't do is pretend the problems don't exist and knowingly leave them broken. I'm merging this now happily knowing that we aren't aware of any regressions. I thank everyone who helped out in making sure of that. |
|
Created with
brew bump-formula-pr
.resource
blocks may require updates.