From: Russ Cox <rsc@golang.org>
Date: Tue, 31 Oct 2017 19:42:17 +0000 (-0400)
Subject: cmd/link: do not store compilation directory in DWARF info
X-Git-Tag: go1.10beta1~489
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=2f170520aa460561997636dec66f303ee69ab684;p=gostls13.git

cmd/link: do not store compilation directory in DWARF info

This makes 'go install cmd/compile' in one directory produce
a different binary from running it in another directory,
which is problematic for reproducible builds.

Change-Id: If26685d2e45d2695413b472142b49694716575fa
Reviewed-on: https://go-review.googlesource.com/74790
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: David Crawshaw <crawshaw@golang.org>
---

diff --git a/src/cmd/link/internal/ld/dwarf.go b/src/cmd/link/internal/ld/dwarf.go
index f92169b30f..d0fae2c79f 100644
--- a/src/cmd/link/internal/ld/dwarf.go
+++ b/src/cmd/link/internal/ld/dwarf.go
@@ -20,7 +20,6 @@ import (
 	"cmd/link/internal/sym"
 	"fmt"
 	"log"
-	"os"
 	"strings"
 )
 
@@ -1024,9 +1023,8 @@ func putpclcdelta(linkctxt *Link, ctxt dwarf.Context, s *sym.Symbol, deltaPC uin
  */
 
 func getCompilationDir() string {
-	if dir, err := os.Getwd(); err == nil {
-		return dir
-	}
+	// OS X requires this, but it's really none of its business.
+	// Hard-code "/" for reproducible builds.
 	return "/"
 }