]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go/internal/envcmd: report PKG_CONFIG after the CGO group
authorJaana Burcu Dogan <jbd@google.com>
Sat, 18 Feb 2017 23:07:13 +0000 (15:07 -0800)
committerJaana Burcu Dogan <jbd@google.com>
Sun, 19 Feb 2017 08:04:38 +0000 (08:04 +0000)
Before the change, `go env` reports PKG_CONFIG in between the
CGO env group:

    GOARCH="amd64"
    GOBIN=""
    GOEXE=""
    GOHOSTARCH="amd64"
    GOHOSTOS="darwin"
    GOOS="darwin"
    GOPATH="/Users/jbd"
    GORACE=""
    GOROOT="/Users/jbd/go"
    GOTOOLDIR="/Users/jbd/go/pkg/tool/darwin_amd64"
    GCCGO="gccgo"
    CC="clang"
    GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lq/qcn67khn4_1b41_g48x3zchh005d21/T/go-build184491598=/tmp/go-build -gno-record-gcc-switches -fno-common"
    CXX="clang++"
    CGO_ENABLED="1"
    PKG_CONFIG="pkg-config"
    CGO_CFLAGS="-g -O2"
    CGO_CPPFLAGS=""
    CGO_CXXFLAGS="-g -O2"
    CGO_FFLAGS="-g -O2"
    CGO_LDFLAGS="-g -O2"

The change makes PKG_CONFIG to be reported as the final item,
and not breaking the CGO_* group apart.

Change-Id: I1e7ed6bdec83009ff118f85c9f0f7b78a67fdd76
Reviewed-on: https://go-review.googlesource.com/37228
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/go/internal/envcmd/env.go

index e8c58917245a65c1390018851f36e651e0d3c40c..2f19d6599203a81af6c4e58ff446d394fae83127 100644 (file)
@@ -94,12 +94,12 @@ func ExtraEnvVars() []cfg.EnvVar {
        b.Init()
        cppflags, cflags, cxxflags, fflags, ldflags := b.CFlags(&load.Package{})
        return []cfg.EnvVar{
-               {Name: "PKG_CONFIG", Value: b.PkgconfigCmd()},
                {Name: "CGO_CFLAGS", Value: strings.Join(cflags, " ")},
                {Name: "CGO_CPPFLAGS", Value: strings.Join(cppflags, " ")},
                {Name: "CGO_CXXFLAGS", Value: strings.Join(cxxflags, " ")},
                {Name: "CGO_FFLAGS", Value: strings.Join(fflags, " ")},
                {Name: "CGO_LDFLAGS", Value: strings.Join(ldflags, " ")},
+               {Name: "PKG_CONFIG", Value: b.PkgconfigCmd()},
        }
 }