]> Cypherpunks.ru repositories - gostls13.git/commitdiff
Revert "cmd/dist: translate /private/var to /var on darwin builders"
authorRuss Cox <rsc@golang.org>
Sat, 28 Oct 2017 11:53:38 +0000 (11:53 +0000)
committerRuss Cox <rsc@golang.org>
Sat, 28 Oct 2017 11:53:49 +0000 (11:53 +0000)
This reverts commit 4f2ee4997470c84a55d89d097e6b8fc45680978f.

Reason for revert: broke mobile builders.

Change-Id: I9fd3ef777ce6401c0c28b03f1dc53ddcdbef5111
Reviewed-on: https://go-review.googlesource.com/74170
Reviewed-by: Russ Cox <rsc@golang.org>
src/cmd/dist/build.go

index cc0c9af6439d6184768997478c96c7a187d035a7..2d4b575a834495e8bf474d8a3c25c88e7dc94284 100644 (file)
@@ -12,7 +12,6 @@ import (
        "os"
        "os/exec"
        "path/filepath"
-       "runtime"
        "sort"
        "strings"
        "sync"
@@ -101,22 +100,6 @@ func xinit() {
        }
        goroot = filepath.Clean(b)
 
-       if runtime.GOOS == "darwin" && strings.HasPrefix(goroot, "/private/") {
-               // The builders don't set $PWD correctly during make.bash
-               // but then they apparently do set it or perhaps $GOROOT
-               // during run.bash. During make.bash we infer that
-               // GOROOT=/private/var/blah/blah but then during run.bash
-               // apparently GOROOT=/var/blah/blah. This makes all commands
-               // seem out of date, which breaks some tests.
-               // Instead of finding the problem in the builders, fix it here.
-               // This is not great but is the best we can do today.
-               f1, err1 := os.Stat(goroot)
-               f2, err2 := os.Stat(strings.TrimPrefix(goroot, "/private"))
-               if err1 == nil && err2 == nil && os.SameFile(f1, f2) {
-                       goroot = strings.TrimPrefix(goroot, "/private")
-               }
-       }
-
        b = os.Getenv("GOROOT_FINAL")
        if b == "" {
                b = goroot