go, branch go1.5beta1

cmd/go: disable vendoredImportPath for code outside $GOPATH

2015-07-07T21:53:11Z

It was crashing. This fixes the build for GO15VENDOREXPERIMENT=1 go test -short runtime Fixes #11416. Change-Id: I74a9114cdd8ebafcc9d2a6f40bf500db19c6e825 Reviewed-on: https://go-review.googlesource.com/11964 Reviewed-by: Russ Cox

cmd/compile: allow static init for unsafe.Pointer(&x) where x is global

2015-07-07T21:51:57Z

This avoids both a write barrier and then dynamic initialization globals of the form var x something var xp = unsafe.Pointer(&x) Using static initialization avoids emitting a relocation for &x, which helps cgo. Fixes #9411. Change-Id: I0dbf480859cce6ab57ab805d1b8609c45b48f156 Reviewed-on: https://go-review.googlesource.com/11693 Reviewed-by: Austin Clements Run-TryBot: Russ Cox

net/http: fix MaxBytesReader at EOF

2015-07-07T21:33:14Z

Fixes #10884 Change-Id: I7cab3c96548867612f579d2cd4ec736309787443 Reviewed-on: https://go-review.googlesource.com/11961 Reviewed-by: Andrew Gerrand Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot

net/http: revert overly-strict part of earlier smuggling defense

2015-07-07T21:33:06Z

The recent https://golang.org/cl/11810 is reportedly a bit too aggressive. Apparently some HTTP requests in the wild do contain both a Transfer-Encoding along with a bogus Content-Length. Instead of returning a 400 Bad Request error, we should just ignore the Content-Length like we did before. Change-Id: I0001be90d09f8293a34f04691f608342875ff5c4 Reviewed-on: https://go-review.googlesource.com/11962 Reviewed-by: Andrew Gerrand Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot

cmd/compile: initialize line number properly for temporaries

2015-07-07T21:30:59Z

The expansion of structure, array, slice, and map literals does not use the right line number in its introduced assignments to temporaries, which leads to incorrect line number attribution for expressions in those literals. Inlining also incorrectly replaced the line numbers of args to inlined functions. This was revealed in CL 9721 because a now-avoided temporary assignment introduced the correct line number. I.e. before CL 9721 "tmp_wrongline := expr" was transformed to "tmp_rightline := expr; tmp_wrongline := tmp_rightline" Also includes a repair to CL 10334 involving line numbers where a spurious -1 remained (should have been 0, now is 0). Fixes #11400. Change-Id: I3a4687efe463977fa1e2c996606f4d91aaf22722 Reviewed-on: https://go-review.googlesource.com/11730 Run-TryBot: David Chase TryBot-Result: Gobot Gobot Reviewed-by: Sameer Ajmani Reviewed-by: Russ Cox

runtime: randomize scheduling in -race mode

2015-07-07T21:27:38Z

Basic randomization of goroutine scheduling for -race mode. It is probably possible to do much better (there's a paper linked in the issue that I haven't read, for example), but this suffices to introduce at least some unpredictability into the scheduling order. The goal here is to have _something_ for Go 1.5, so that we don't start hitting more of these scheduling order-dependent bugs if we change the scheduler order again in Go 1.6. For #11372. Change-Id: Idf1154123fbd5b7a1ee4d339e93f97635cc2bacb Reviewed-on: https://go-review.googlesource.com/11795 Reviewed-by: Austin Clements

cmd/go: ignore volume name case when checking vendor path

2015-07-07T21:02:17Z

Fixes #11409 Change-Id: Ic1610e124b2d8b2b12310fc9538d5078cc7302a0 Reviewed-on: https://go-review.googlesource.com/11316 Reviewed-by: Russ Cox

cmd/go: create executable when installing to working directory

2015-07-07T21:00:44Z

Fixes #11065. Change-Id: Idd854facd5fa78c0334f86740f351d404f9a5b2d Reviewed-on: https://go-review.googlesource.com/11511 Reviewed-by: Russ Cox

cmd/go: do not panic on invalid import path containing "/vendor/"

2015-07-07T21:00:01Z

Fixes #11414 Change-Id: I45a41b98554f00362d9222e9c68a441dbfc23cb8 Reviewed-on: https://go-review.googlesource.com/11700 Reviewed-by: Russ Cox

net/http/httputil: make ReverseProxy support Trailers

2015-07-07T14:30:49Z

Go's continuous build system depends on HTTP trailers for the buildlet interface. Andrew rewrote the makerelease tool to work in terms of Go's builder system (now at x/build/cmd/release), but it previously could only create GCE-based buildlets, which meant x/build/cmd/release couldn't build the release for Darwin. https://golang.org/cl/11901 added support for proxying buildlet connections via the coordinator, but that exposed the fact that httputil.ReverseProxy couldn't proxy Trailers. A fork of that code also wasn't possible because net/http needlessly deleted the "Trailer" response header in the Transport code. This mistake goes back to "release-branch.r56" and earlier but was never noticed because nobody ever uses Trailers, and servers via ResponseWriter never had the ability to even set trailers before this Go 1.5. Note that setting trailers requires pre-declaring (in the response header) which trailers you'll set later (after the response body). Because you could never set them, before this release you could also never proxy them. Change-Id: I2410a099921790dcd391675ae8610300efa19108 Reviewed-on: https://go-review.googlesource.com/11940 Reviewed-by: Andrew Gerrand Run-TryBot: Brad Fitzpatrick