]> Cypherpunks repositories - gostls13.git/commitdiff
misc/dashboard/builder: always do a native build
authorDave Cheney <dave@cheney.net>
Thu, 18 Apr 2013 06:42:56 +0000 (16:42 +1000)
committerDave Cheney <dave@cheney.net>
Thu, 18 Apr 2013 06:42:56 +0000 (16:42 +1000)
https://golang.org/cl/8134043 disabled cgo when cross compiling, this means builders which compile for both amd64 and 386 will be compiling the latter with cgo disabled.

This proposal modifies the builder to mirror the dist tool by always doing a native build.

Tested on my darwin/amd64 builder and confirmed the result when building darwin/386 is a native 386 build with cgo enabled.

R=bradfitz, dsymonds, r, adg
CC=golang-dev
https://golang.org/cl/8842044

misc/dashboard/builder/main.go

index 17785784253d8fcd2260aa87bee1cb1efb3370e8..9a155bd0c6706cf6832186750a1294ab9dc980c0 100644 (file)
@@ -33,8 +33,6 @@ const (
 var extraEnv = []string{
        "CC",
        "GOARM",
-       "GOHOSTARCH",
-       "GOHOSTOS",
        "PATH",
        "TMPDIR",
        "USER",
@@ -409,7 +407,9 @@ func (b *Builder) envv() []string {
        }
        e := []string{
                "GOOS=" + b.goos,
+               "GOHOSTOS=" + b.goos,
                "GOARCH=" + b.goarch,
+               "GOHOSTARCH=" + b.goarch,
                "GOROOT_FINAL=/usr/local/go",
        }
        for _, k := range extraEnv {
@@ -424,7 +424,9 @@ func (b *Builder) envv() []string {
 func (b *Builder) envvWindows() []string {
        start := map[string]string{
                "GOOS":         b.goos,
+               "GOHOSTOS":     b.goos,
                "GOARCH":       b.goarch,
+               "GOHOSTARCH":   b.goarch,
                "GOROOT_FINAL": `c:\go`,
                "GOBUILDEXIT":  "1", // exit all.bat with completion status.
        }