diff options
| author | Dmitri Shuralyov <dmitshur@golang.org> | 2023-07-29 16:33:13 -0400 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2023-08-01 22:28:15 +0000 |
| commit | f719de079871f51998e735fc51b3891d50217262 (patch) | |
| tree | d50018850c014d0a00425034b60458ab40d321ce /src | |
| parent | f32e0c30e4a002cda2dbb5e628a7f1047e304e01 (diff) | |
| download | go-f719de079871f51998e735fc51b3891d50217262.tar.xz | |
cmd/dist: tolerate macOS writing .DS_Store to GOROOT/bin
I was trying out gorebuild (a program that runs make.bash many times)
on a macOS system. Unfortunately there were a few failed invocations
on my first try, but not with a very good or interesting reason:
go tool dist: unexpected new file in $GOROOT/bin: .DS_Store
Tolerate it since it's not unexpected, and will not affect the build.
Change-Id: I656536b896098c2ba934667196d4ce82e706c8da
Reviewed-on: https://go-review.googlesource.com/c/go/+/513763
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/cmd/dist/build.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/cmd/dist/build.go b/src/cmd/dist/build.go index 8973a87168..0afa5f062e 100644 --- a/src/cmd/dist/build.go +++ b/src/cmd/dist/build.go @@ -1577,6 +1577,9 @@ func cmdbootstrap() { ok[f] = true } for _, f := range binFiles { + if gohostos == "darwin" && filepath.Base(f) == ".DS_Store" { + continue // unfortunate but not unexpected + } elem := strings.TrimSuffix(filepath.Base(f), ".exe") if !ok[f] && elem != "go" && elem != "gofmt" && elem != goos+"_"+goarch { fatalf("unexpected new file in $GOROOT/bin: %s", elem) |
