]> Cypherpunks repositories - gostls13.git/commit
cmd/go: fix cgo test when min macOS version is set
authorJay Conrod <jayconrod@google.com>
Fri, 24 Jan 2020 16:25:52 +0000 (08:25 -0800)
committerJay Conrod <jayconrod@google.com>
Fri, 24 Jan 2020 17:36:44 +0000 (17:36 +0000)
commit1f9f88b95eaec50c72c8595ca9f52b7b876e28f9
tree3ebe6c091edb0a09609a68f0354fce26c730f7e6
parent8d7be1e3c9a98191f8c900087025c5e78b73d962
cmd/go: fix cgo test when min macOS version is set

Regression tests for #24161 use a macro to conditionally compile some
stub definitions. The macro tests that the minimum macOS version is
less than 10.12.

We get duplicate definitions when building this test with
CGO_CFLAGS=-mmacosx-version-min=10.x where 10.x < 10.12. With this
change, we use a different macro, __MAC_OS_X_VERSION_MAX_ALLOWED__,
which tests the SDK version instead of the minimum macOS version. This
checks whether these definitions are present in headers.

After this change, 'go tool dist test cgo_test' should pass with
CGO_FLAGS=-mmacosx-version-min=10.10.

Updates #35459

Change-Id: I88d63601c94b0369c73c38d216a2d41ba7d4e579
Reviewed-on: https://go-review.googlesource.com/c/go/+/216243
Run-TryBot: Jay Conrod <jayconrod@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
misc/cgo/test/testdata/issue24161e0/main.go
misc/cgo/test/testdata/issue24161e1/main.go
misc/cgo/test/testdata/issue24161e2/main.go