- 03 Oct, 2017 16 commits
-
-
Lynn Boger authored
Some tests in misc/cgo/test are run with various options including '-linkmode=external "-extldflags=-pie"'. On ppc64x passing -pie to the external linker with code that was not compiled as position independent is incorrect. This works by luck in many cases but is not guaranteed to work. I suspect it is an issue on other targets as well. This will now run the tests using -buildmode=pie for the platforms that support that buildmode option. Fixes #21954 Change-Id: I25fc7573f2d3cb5b0d1c691a0ac91aef7715404f Reviewed-on: https://go-review.googlesource.com/66870 Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Tim Cooper authored
Previously, only the empty interface could be formatted to print on a single line. This behaviour made short one-method interfaces in function definitions and type assertions more verbose than they had to be. For example, the following type assertion: if c, ok := v.(interface { Close() error }); ok { } Can now be formatted as: if c, ok := v.(interface{ Close() error }); ok { } Fixes #21952 Change-Id: I896f796c5a30b9f4da2be3fe67cb6fea5871b835 Reviewed-on: https://go-review.googlesource.com/66130 Run-TryBot: Robert Griesemer <gri@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
-
Joe Tsai authored
s/TypeSymLink/TypeSymlink/g Change-Id: I2550843248eb27d90684d0036fe2add0b247ae5a Reviewed-on: https://go-review.googlesource.com/67810Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
griesemer authored
Apply gofmt to src, misc. Fixes #22111. Change-Id: Ib1bda0caaf2c1787a8137b7a61bbef7a341cc68c Reviewed-on: https://go-review.googlesource.com/67633 Run-TryBot: Robert Griesemer <gri@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Matthew Dempsky <mdempsky@google.com>
-
Giovanni Bajo authored
High Sierra has a new commpage layout (this is issue #3188), so we need to adjust the code to handle multiple versions of the layout. In preparation for this change, we rename the existing offset macros with a prefix that identifies the commpage version they refer to. Updates #22037 Change-Id: Idca4b7a855a2ff6dbc434cd12453fc3194707aa8 Reviewed-on: https://go-review.googlesource.com/67331 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Florian Uekermann authored
Loading location data from tzdata files was only supported from default paths on android. This change enables support on all OS via the ZONEINFO environment variable and reduces the amount of android specific code significantly. Furthermore, unsuccessful calls to LoadLocation now return the first error encountered, including errors from attempting to load a location from the source specified by ZONEINFO. Errors indicating that the source or location was not found are ignored until all possible sources have been traversed. Change-Id: I45bc23b92253c9447f12f95f3ca29a7e613ed995 Reviewed-on: https://go-review.googlesource.com/67170Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Ian Lance Taylor authored
Fixes #22077 Change-Id: I8d1b18e78719514fa2a410e85c9d3934c47cd2fe Reviewed-on: https://go-review.googlesource.com/66930Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>
-
Grant Griffiths authored
Fixes #21159. Change-Id: I2c5ad505d673e213a548e5d632a5b3ad706e0dde Reviewed-on: https://go-review.googlesource.com/67635 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Elias Naur authored
The lldb child process is killed if a test runs too long. Also kill it when the setup times out (and is retried). Might help with builder flakes where all 5 attempts to start up lldb fail even though the tests before and after the timeouts succeed. For example: ... ok vendor/golang_org/x/net/route 37.797s lldb setup error: command timeout (lldb start for 17s) start timeout, trying again lldb setup error: command timeout (lldb start for 17s) start timeout, trying again lldb setup error: command timeout (lldb start for 17s) start timeout, trying again lldb setup error: command timeout (lldb start for 17s) start timeout, trying again lldb setup error: command timeout (lldb start for 17s) go_darwin_arm_exec: failed to start test harness (retry attempted) FAIL vendor/golang_org/x/text/transform 115.185s ok vendor/golang_org/x/text/unicode/norm 122.773s ... Change-Id: I6638860522896491dccfa12f1e520c0f23df6d66 Reviewed-on: https://go-review.googlesource.com/67791 Run-TryBot: Elias Naur <elias.naur@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
The point of this code is to print a warning about repeated go test invocations rebuilding the same packages over and over. The new cache will eliminate this failure mode and with it the need for the warning and this field. Change-Id: Ied79b3ca67d51a61f44629de6ae4974e6c8dd5a9 Reviewed-on: https://go-review.googlesource.com/56282Reviewed-by: David Crawshaw <crawshaw@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Mistake introduced just before submitting CL 67650 that somehow did not break a final pre-submit all.bash on my laptop. Not sure why all.bash passes locally when mkdeps.go doesn't build. I guess the test only runs on builders? Change-Id: I18fb91ada47605035345ba4b2f5e360a5c4b7f6e Reviewed-on: https://go-review.googlesource.com/67850 Run-TryBot: Russ Cox <rsc@golang.org> Reviewed-by: Russ Cox <rsc@golang.org>
-
Russ Cox authored
In addition to the obvious formatting change, this also drops from deps.go any indirect dependencies, so that when you add a new import to one package, the resulting diff only affects that one package, not every package that imports that package directly or indirectly. That makes the file a bit easier to understand, if you need to debug it or deal with a possible merge conflict. The code to trim the import lists (but not too much) was more than I wanted to do in shell, so I rewrote mkdeps in Go. The shell script is left behind for backwards-compatibility with people who have learned to run ./mkdeps.bash (or documentation suggesting the same). Change-Id: I0bf27b5b27d0440e11ea830b00735c73f58eae03 Reviewed-on: https://go-review.googlesource.com/67650Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
Austin Clements authored
This method indicates whether a type contains any *heap* pointers, not just whether it contains any pointers. Rename the method to make this clear. Change-Id: Ifff143e2f02a820444ac26b84250495c0098cb33 Reviewed-on: https://go-review.googlesource.com/67690 Run-TryBot: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
-
Lynn Boger authored
A new testcase TestSelectFairness was recently added, and since then the ppc64le build tests have intermittently failed. This adds a change to skip this test on ppc64le using SkipFlaky to help determine if the problem is with the test or something else with that commit. Updates #22047 Change-Id: Idfef72ed791c5bd45c42ff180947fea3df280ea7 Reviewed-on: https://go-review.googlesource.com/67631 Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Keith Randall authored
Use entire inlining call stack to decide whether two panic calls can be merged. We used to merge panic calls when only the leaf line numbers matched, but that leads to places higher up the call stack being merged incorrectly. Fixes #22083 Change-Id: Ia41400a80de4b6ecf3e5089abce0c42b65e9b38a Reviewed-on: https://go-review.googlesource.com/67632 Run-TryBot: Emmanuel Odeke <emm.odeke@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
-
Martin Kreichgauer authored
Fixes #21615 Change-Id: Ic13190617d9b446b35f5dd00f142597c187ab669 Reviewed-on: https://go-review.googlesource.com/67270Reviewed-by: Adam Langley <agl@golang.org> Reviewed-by: Martin Kreichgauer <martinkr@google.com> Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
- 02 Oct, 2017 9 commits
-
-
Artyom Pervukhin authored
Fixes #22084. Change-Id: If405ffdc57fcf81de3c0e8473c45fc504db735bc Reviewed-on: https://go-review.googlesource.com/67410 Run-TryBot: Tom Bergan <tombergan@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Tom Bergan <tombergan@google.com>
-
Adam Langley authored
We'll need this for handling name constraints during verification. Change-Id: I4ef19d9489fb2a9ae9a62699d81cef92a21fda28 Reviewed-on: https://go-review.googlesource.com/62692 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
Adam Langley authored
Change-Id: Idcda0fc1607157cb5bbf0521fbdc0c77f043ca3a Reviewed-on: https://go-review.googlesource.com/62691 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
Alan Donovan authored
+ Test. Change-Id: I42eaea1c716217f7945c008ff4bde6de14df5687 Reviewed-on: https://go-review.googlesource.com/34619Reviewed-by: Robert Griesemer <gri@golang.org>
-
Guilherme Rezende authored
Change-Id: Ic3f7f575d3640706adb7d64545ed8027add6c58f Reviewed-on: https://go-review.googlesource.com/61350 Run-TryBot: Tom Bergan <tombergan@google.com> Reviewed-by: Tom Bergan <tombergan@google.com>
-
Alberto Donizetti authored
Fixes #9021 Change-Id: Id4465857c765342aa8c4b5bc2fdc06754a290025 Reviewed-on: https://go-review.googlesource.com/67530Reviewed-by: Cherry Zhang <cherryyz@google.com> Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Russ Cox authored
These are no longer used. Change-Id: I17fa29bbf90b1a41109fde4504eeff06af1eb8be Reviewed-on: https://go-review.googlesource.com/56281 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Dave Cheney <dave@cheney.net>
-
Elias Naur authored
After switching to an iPhone 5 for the darwin/arm builds, TestStoreLoadRelAcq32 started to timeout on every builder run. Adding the same memory barriers as armLoadUint64 and armStoreUint64 makes the test complete successfully. Fixes sync/atomic tests on the darwin/arm builder. Change-Id: Id73de31679304e259bdbd7f2f94383ae7fd70ee4 Reviewed-on: https://go-review.googlesource.com/67390 TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
-
Tim Cooper authored
Fixes #21950 Change-Id: I6fa392abd2c3bf6a4f80f14c6b1419470e9a944d Reviewed-on: https://go-review.googlesource.com/66750 Run-TryBot: Rob Pike <r@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rob Pike <r@golang.org>
-
- 01 Oct, 2017 2 commits
-
-
Dave Cheney authored
Move some local declarations closer to their use, reducing their respective lifetimes. Spotted while reviewing CL 67318. Change-Id: I68db67fe8530344d95e50efb6587bc724e1171f5 Reviewed-on: https://go-review.googlesource.com/67370 Run-TryBot: Dave Cheney <dave@cheney.net> Reviewed-by: Marvin Stenger <marvin.stenger94@gmail.com> Reviewed-by: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Cyrill Schumacher authored
A new switch case for converting the source string type into a destination RawBytes type avoids the reflection based conversion. Speed up from old ~61.7ns/op down to ~49ns/op. A second new switch case allows to convert and assign a source time.Time type into a destination sql.RawBytes type. This switch case appends the time to the reset RawBytes slice. This allows the reuse of RawBytes and avoids allocations. Fixes #20746 Change-Id: Ib0563fd5c5c7cb6d9d0acaa1d9aa7b2927f1329c Reviewed-on: https://go-review.googlesource.com/66830 Run-TryBot: Daniel Theophanes <kardianos@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Daniel Theophanes <kardianos@gmail.com>
-
- 30 Sep, 2017 4 commits
-
-
Adam Langley authored
Previously, any “explicit” and/or “tag” decorations on a RawValue would be ignored when unmarshaling. The RawValue would swallow whatever element was encountered. This change causes these decorations to be respected. Thus a field like: Foo asn1.RawValue `asn1:"explicit,tag:1,optional"` will only match if an explicit tag with value one is encountered. Otherwise the RawValue will get the default value and parsing will move onto the next element. Thanks to Martin Kreichgauer for reporting the issue. Change-Id: If6c4488685b9bd039cb5e352d6d75744f98dbb1f Reviewed-on: https://go-review.googlesource.com/34503 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Adam Langley <agl@golang.org>
-
Russ Cox authored
Now that we have -importcfg, there's no need for the temporary directory trees that mirror the import path structure, and we can drop a bunch of complex code that was building and maintaining that structure. This should fix "file name too long" errors on systems with low limits. (For example #10651 and #17070, although we fixed those by adding code to deal with very long file names on Windows instead.) Change-Id: I11e221c6c1edeb81c3b2f1d89988f5235aa2bbb9 Reviewed-on: https://go-review.googlesource.com/56280Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Until now the subdirectories under $WORK have had predictable names, so it was OK to strip just $WORK from the file names that end up in object files. In the future, those predictable names would cause predictable collisions when compiling one package in two different ways, so we're moving toward arbitrary, unpredictable subdirectory names instead. When we do that, if the names appear in the object files we won't get reproducible builds. Take the subdirectory names out now, to make the later change safe. Change-Id: I8057d1cc73f6e35c98b7718c9789c161dcbd87c0 Reviewed-on: https://go-review.googlesource.com/67251 Run-TryBot: Russ Cox <rsc@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Russ Cox authored
CL 64793 removed the collect step, which took all the generated .o files and merged them into a single _all.o. Now the generated .o files all go directly into the final .a. The only property of the _all.o approach that was lost in CL 64793 was that before we could be sure that the one name we used was "ar-compatible", that is, short enough not to be truncated. Now that the generated .o files are being kept directly, this CL gives them guaranteed ar-compatible names. This doesn't matter for nearly all uses today, but for some future processing it might help not to lose the .o suffix or not to end up with two identical entries with truncated names. I might not have bothered with this except that it's what's leftover after syncing my own CL disabling _all.o (necessary for reproducible builds on macOS) against Ian's CL 64793, which landed first. Change-Id: Ic86ed2a51432a5a4c58dc523e092a86d341f1997 Reviewed-on: https://go-review.googlesource.com/67250 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 29 Sep, 2017 7 commits
-
-
David Crawshaw authored
Fixes #18584 Change-Id: I5f9428758999cacee49f3449e596e0a88bc06f91 Reviewed-on: https://go-review.googlesource.com/67150 Run-TryBot: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Kevin Burke authored
Change-Id: Id900636ee58a39aaa3dc1c601cb83706d3e2fbe8 Reviewed-on: https://go-review.googlesource.com/67190Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
I don't know why these tests must import runtime/cgo in _testmain.go, but if they must, they must also tell the rest of the go command that they are doing so. Should fix the newly-broken darwin/arm and darwin/arm64 builders. Change-Id: I9b183f8c84c6f403bf3a90cbfc838d6ef428e16f Reviewed-on: https://go-review.googlesource.com/67230 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
Ian Lance Taylor authored
Modifying the input slice broke the new test for whether gccgo supports -fgo-importcfg, as the test passed a slice of the argument slice it was in the process of building. Fixes #22089 Change-Id: I45444a82673223c46be0c8579da3e31a74c32d73 Reviewed-on: https://go-review.googlesource.com/67191 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Russ Cox <rsc@golang.org>
-
Marvin Stenger authored
nilcheckelim2 cleans up by copying b.Values in a loop, omitting OpUnknowns. However, the common case is that there are no OpUnknowns, in which case we can skip a lot of work. So we track the first nilcheck which was eliminated, if any, and only start copying from there. If no nilcheck was eliminated we won't copy at all. Fixes #20964 Change-Id: Icd44194cf8ac81ce6485ce257b4d33e093003a40 Reviewed-on: https://go-review.googlesource.com/65651 Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
-
Russ Cox authored
Implement importcfg on behalf of gccgo by writing out a tree of symbolic links. In addition to keeping gccgo working with the latest changes, this also fixes a precedence bug in gccgo's cmd/go vendor support (the vendor equivalent of #14271). Change-Id: I0e5645116e1c84c957936baf22e3126ba6b0d46e Reviewed-on: https://go-review.googlesource.com/61731 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
Russ Cox authored
This is a step toward using cached build artifacts: the importcfg will direct the compiler and linker to read them right from the cache if necessary. However, this CL does not have a cache yet, so it still reads them from the usual install location or build location. Even so, this fixes a long-standing issue that -I and -L (no longer used) are not expressive enough to describe complex GOPATH setups. Shared libraries are handled enough that all.bash passes, but there may still be more work to do here. If so, tests and fixes can be added in follow-up CLs. Gccgo will need updating to support -importcfg as well. Fixes #14271. Change-Id: I5c52a0a5df0ffbf7436e1130c74e9e24fceff80f Reviewed-on: https://go-review.googlesource.com/56279 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 28 Sep, 2017 2 commits
-
-
Daniel Martí authored
Add the package to the table and start it off with a few small, basic functions. Inspired by CL 66331, which added flag.ro. Updates #21851. Change-Id: I3995cde1ff7bb09a718110473bed8b193c2232a5 Reviewed-on: https://go-review.googlesource.com/66990 Run-TryBot: Daniel Martí <mvdan@mvdan.cc> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ilya Tocar <ilya.tocar@intel.com> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Tim Cooper authored
Previously, a multi-line flag usage string would not be indented with the rest of the usage strings. This made long usage strings difficult to read. For example, the usage for flag.String("A", "", "1\n2\n3") would be printed as: -A 1 2 3 But will now be printed as: -A 1 2 3 Also fixes a slight error in the FlagSet.PrintDefaults documentation. Fixes #20799 Change-Id: I4379c6b7590fdb93a2809a01046a0f6ae32c3e5d Reviewed-on: https://go-review.googlesource.com/66711 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Russ Cox <rsc@golang.org>
-