From 24f302a69d80f2e89db81fad600165b2f857c193 Mon Sep 17 00:00:00 2001 From: David Symonds Date: Mon, 19 Aug 2013 11:11:27 +1000 Subject: [PATCH] cmd/dist: join with TMPDIR more carefully to avoid // in path. This might fix the mkdtemp problem on the darwin builders if they have TMPDIR set to a path ending in a slash; at worse this will result in cleaner path names. R=golang-dev, r CC=golang-dev https://golang.org/cl/13097043 --- src/cmd/dist/unix.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/cmd/dist/unix.c b/src/cmd/dist/unix.c index 4a4feeb064..0011e2f278 100644 --- a/src/cmd/dist/unix.c +++ b/src/cmd/dist/unix.c @@ -466,7 +466,9 @@ xworkdir(void) xgetenv(&b, "TMPDIR"); if(b.len == 0) bwritestr(&b, "/var/tmp"); - bwritestr(&b, "/go-cbuild-XXXXXX"); + if(b.p[b.len-1] != '/') + bwrite(&b, "/", 1); + bwritestr(&b, "go-cbuild-XXXXXX"); p = bstr(&b); if(mkdtemp(p) == nil) fatal("mkdtemp(%s): %s", p, strerror(errno));