]> Cypherpunks repositories - gostls13.git/commitdiff
crypto/x509: bump minimum macOS version to 10.11
authorTobias Klauser <tklauser@distanz.ch>
Thu, 19 Mar 2020 21:33:40 +0000 (22:33 +0100)
committerFilippo Valsorda <filippo@golang.org>
Thu, 19 Mar 2020 22:33:15 +0000 (22:33 +0000)
The minimum macOS supported version is 10.11 as of Go 1.14, see #23011.
Thus, bump macosx-version-min to 10.11

While at it, drop __MAC_OS_X_VERSION_MAX_ALLOWED as suggested by
Filippo:

  In general, I can see why we'd want to tell the libraries which
  minimum version we target so they drop compatibility with older
  versions. No idea why we'd specify a max version, unless it's to make
  sure we don't use APIs added after that version, but then it would
  have to be 1011 not 1015.

  Let's try dropping that define and see if anything blows up? ¯\_(ツ)_/¯

Change-Id: I6b76623a9404724ccda40311ff95b3475ae8a60c
Reviewed-on: https://go-review.googlesource.com/c/go/+/214059
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Filippo Valsorda <filippo@golang.org>
src/crypto/x509/root_cgo_darwin.go

index 8a54282a6b2d17e4d155327be3f7ca4815d03f05..f0ad8e995890e425035afb30c13da20c0f54e003 100644 (file)
@@ -7,7 +7,7 @@
 package x509
 
 /*
-#cgo CFLAGS: -mmacosx-version-min=10.10 -D__MAC_OS_X_VERSION_MAX_ALLOWED=101300
+#cgo CFLAGS: -mmacosx-version-min=10.11
 #cgo LDFLAGS: -framework CoreFoundation -framework Security
 
 #include <errno.h>