aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike McQuaid2013-10-08 08:39:04 +0100
committerMike McQuaid2013-10-08 08:39:16 +0100
commit1400e08094f0877ad4d079dbc7bfe88acd58bdcc (patch)
tree6cffe4c0265838d68c264838f36b48a2beeb64ef
parentac8712dc21ec995f5404ec9e61c6a7ebece26c74 (diff)
downloadhomebrew-1400e08094f0877ad4d079dbc7bfe88acd58bdcc.tar.bz2
go: build cgo with clang for devel versions.
Closes #23119.
-rw-r--r--Library/Formula/go.rb11
1 files changed, 1 insertions, 10 deletions
diff --git a/Library/Formula/go.rb b/Library/Formula/go.rb
index 2c9868a92..8da5f440d 100644
--- a/Library/Formula/go.rb
+++ b/Library/Formula/go.rb
@@ -23,10 +23,7 @@ class Go < Formula
sha1 '9f39106e06f552e9bf6d15d201c4663c051d4f89'
end
- # the cgo module cannot build with clang
- # NOTE it is ridiculous that we put this stuff in the class
- # definition, it needs to be in a pre-install test function!
- if build.with? 'cgo'
+ if build.with? 'cgo' and not build.devel?
depends_on 'apple-gcc42' if MacOS.version >= :mountain_lion
fails_with :clang do
@@ -96,13 +93,7 @@ class Go < Formula
when $GOPATH and $GOROOT are set to the same value.
More information here: http://golang.org/doc/code.html#GOPATH
-
- FYI: We probably didn't build the cgo module because it doesn't build with
- clang.
EOS
- # NOTE I would have the cgo caveat only show if we didn't build it but the
- # state matrix for that seems inconclusive, ENV.compiler doesn't actually
- # mean for sure that we used that compiler.
end
test do