-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
website: inconsistency in supported OS X versions #16625
Comments
And later down the page, it says the requirements for compiling from source are "Mac OS X 10.7 or later". I seem to recall that 10.7 is the minimum requirement for building for source, but somehow the binaries we produce (or the packaging) requires 10.8. I forget the details. @adg? But the first mistake you cited is easy to fix at least. |
CL https://golang.org/cl/25548 mentions this issue. |
Updates #16625 Change-Id: Icac6705828bd9b29379596ba64b34d922b9002c3 Reviewed-on: https://go-review.googlesource.com/25548 Reviewed-by: Ian Lance Taylor <iant@golang.org>
I kept this bug open (using "Updates" in the commit message) so we can figure out what the story with source vs binary release supported versions is on OS X. I was waiting for @adg. |
Okay, I'll tag this for 1.7 then, hopefully we can get the docs right before we release. |
I don't remember anymore. |
I'll try a build on OS X 10.7 and see what happens. |
Updates golang#16625 Change-Id: Icac6705828bd9b29379596ba64b34d922b9002c3 Reviewed-on: https://go-review.googlesource.com/25548 Reviewed-by: Ian Lance Taylor <iant@golang.org>
Just tried OS X 10.7. It doesn't even compile:
I'm sure we can work around it, but there's no point. And we don't want to run builders for OS X 10.{7,8,9,10,11,12}. Too many. 8 is old enough. Plus I still recall that 7 (Lion) leaked pipes in exec or something. I'll fix the docs. |
CL https://golang.org/cl/28870 mentions this issue. |
https://golang.org/dl/ mentions:
https://golang.org/doc/install says:
The text was updated successfully, but these errors were encountered: