Note: CLs won't be assigned to anyone automatically
because oscarTeam is not a gerrit account.
Change-Id: Ie20c419b3caf310fcc0da373fe9f63415629abec
Reviewed-on: https://go-review.googlesource.com/c/build/+/623639
Reviewed-by: Tatiana Bradley <tatianabradley@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Otherwise deploying runs into:
Step 5/15 : RUN go mod download
---> Running in 2cb572f3515a
go: go.mod requires go >= 1.22.0 (running go 1.21.13; GOTOOLCHAIN=local)
Change-Id: I70e4df2a3f9db92cb144351d4b67a40cfc7fa482
Reviewed-on: https://go-review.googlesource.com/c/build/+/612557
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Tim King <taking@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
There have been several moves recently.
Updates golang/go#65355.
Change-Id: Ie00f2e540bc7e51d1f833dffb587b3ce9954601f
Reviewed-on: https://go-review.googlesource.com/c/build/+/595295
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Michael Pratt <mpratt@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
It's not needed as of Go 1.20, the generator
is seeded randomly at program startup anyway.
Change-Id: Ic1f5cd84df333a73c8bd39be70f88df03c690e81
Reviewed-on: https://go-review.googlesource.com/c/build/+/599736
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
By Rob's request.
Change-Id: Ic8bcb7aeb95606dbe4f93b65a564808552ab838c
Reviewed-on: https://go-review.googlesource.com/c/build/+/598475
Reviewed-by: Rob Pike <r@golang.org>
Auto-Submit: Russ Cox <rsc@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Change-Id: I205ec67c50a303b60220c709e3c5f9687b7fa491
Reviewed-on: https://go-review.googlesource.com/c/build/+/596477
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Than McIntosh <thanm@google.com>
My last day at Google will be March 15,
so I will no longer be on the Go team at that point.
Change-Id: Ic851359719850ffe1799da30b7394ea021998dc0
Reviewed-on: https://go-review.googlesource.com/c/build/+/570235
Reviewed-by: Damien Neil <dneil@google.com>
Commit-Queue: Bryan Mills <bcmills@google.com>
Reviewed-by: Sam Thanawalla <samthanawalla@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
Reviewed-by: Michael Matloob <matloob@golang.org>
The new slices.SortFunc function is clearer and faster.
Change-Id: Id1194dc7fb64cdcbe53ad7faa57f7dfb40e533c3
Reviewed-on: https://go-review.googlesource.com/c/build/+/565575
Commit-Queue: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
The only use was removed in CL 163218 (the removed href for
.Owner.Name in reviews.tmpl).
It's unlikely that anybody is using the field manually as you have to
match the owner name, not e-mail address.
Change-Id: I1209680a7f729fa743050872c89beb52a4fe19ac
Reviewed-on: https://go-review.googlesource.com/c/build/+/565555
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Commit-Queue: Ian Lance Taylor <iant@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Use a team rather than individuals, so we can more easily CC folks who
are currently working on telemetry.
Change-Id: I91a58404f11a350e4b84bd433e03186eade7772e
Reviewed-on: https://go-review.googlesource.com/c/build/+/562247
Reviewed-by: Peter Weinberger <pjw@google.com>
Reviewed-by: Michael Matloob <matloob@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Use a newer supported version.
For golang/go#64169.
Change-Id: Ice8fea55951bf1615a04aca24a870556c281877a
Reviewed-on: https://go-review.googlesource.com/c/build/+/556359
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
and clean up unused names
Change-Id: If9b7645024903f3cbb78f66b672e00513d7b2fb5
Reviewed-on: https://go-review.googlesource.com/c/build/+/554015
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Hyang-Ah Hana Kim <hyangah@gmail.com>
Given my contributions to the atomic apis, I would like to volunteer as
a secondary maintainer for the sync/atomic and runtime/internal/atomic
packages.
Change-Id: Id47b71d04a2e91d38e3c1aabbb33ad743e17b608
GitHub-Last-Rev: cab3ed7cd3
GitHub-Pull-Request: golang/build#82
Reviewed-on: https://go-review.googlesource.com/c/build/+/548376
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
I assume nobody wants to individually volunteer for goimports and such,
even though they do occasionally get CLs :)
Change-Id: Id50f265be13f0dd96924ccad34a282718528c6dd
Reviewed-on: https://go-review.googlesource.com/c/build/+/544635
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Heschi Kreinick <heschi@google.com>
Reviewed-by: Robert Findley <rfindley@google.com>
This is helpful to see activity by path, and discover which paths could benefit from additional owners.
Change-Id: Ic7682b68f13e60155587d52f043d0594ab2b4e50
GitHub-Last-Rev: a06f6634ab
GitHub-Pull-Request: golang/build#52
Reviewed-on: https://go-review.googlesource.com/c/build/+/460117
Reviewed-by: Michael Pratt <mpratt@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Previously the calculation of open issues over previous days only took
into account when the issue was created and closed, but not that issues
may have been added to or removed from the current development milestone
over their lifetime.
This was especially noticeable when relui moved 100 open issues from
Go1.21 to Go1.22, yet the burndown graph made it seem like there had
always been around 225 issues in it, pretty misleading. :( maintner
already tracks milestoned and demilestoned events along with event
creation time, so take them into account to de-gaslight the graph.
Change-Id: Iba3f3f68e4ddc140d5612317c02bdfefe4693d8d
Reviewed-on: https://go-review.googlesource.com/c/build/+/517716
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Update some random stuff that hasn't been deployed for a while.
Change-Id: Ib8b7059e770d1176b8b8885c479019e404fa97d0
Reviewed-on: https://go-review.googlesource.com/c/build/+/509896
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Heschi Kreinick <heschi@google.com>
Auto-Submit: Heschi Kreinick <heschi@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Change-Id: I975306ddf1a22db5c96f5588075abdae27ddf662
Reviewed-on: https://go-review.googlesource.com/c/build/+/507296
Auto-Submit: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Eli Bendersky <eliben@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Change-Id: Ib0865f4ba154707662503cd106afe490745f085f
Reviewed-on: https://go-review.googlesource.com/c/build/+/499375
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Peter Weinberger <pjw@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Hyang-Ah Hana Kim <hyangah@gmail.com>
This package has more to do with runtime/pprof than net/http.
Change-Id: I69ecfedc1d425c9c7f144d45d14c4b69674d1d5f
Reviewed-on: https://go-review.googlesource.com/c/build/+/482338
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
For golang/go#58141.
Change-Id: I64473f26a693b982b891b4e11dae11a28901fca4
Reviewed-on: https://go-review.googlesource.com/c/build/+/479735
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Run-TryBot: Michael Pratt <mpratt@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Replace "Go Authors" with "The Go Authors." to be consistent with the
standard copyright header at https://go.dev/doc/contribute#copyright.
There were also a couple instances of a double space and double "the".
Change-Id: Ife7a80dd40f1a760ba85cbfee270356d46cfc53f
Reviewed-on: https://go-review.googlesource.com/c/build/+/472376
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Change-Id: I15769c4302e8dd1d649e0eaa9b16099f0eda603b
Reviewed-on: https://go-review.googlesource.com/c/build/+/464239
Run-TryBot: Ian Lance Taylor <iant@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
We know which teams generally own some of these golang.org/x repos,
so document that here for better visibility. Similarly, add relevant
teams to the embed, hash, html, and unicode directories in the main
Go repo.
Also remove out-of-date branch and domain entries.
Change-Id: Id9c564a7b365972f3630b2ac77448eb64b48ef7b
Reviewed-on: https://go-review.googlesource.com/c/build/+/457340
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Robert Findley <rfindley@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Remove some of my no-longer-relevant entries, since their presence is
causing more confusion than benefit.
x/tools/go/vcs is hopefully heading towards deprecation (see proposal
go.dev/issue/57051) and in general it's outside of scope of what I own
by now.
Similarly, x/tools/cmd/godoc is already deprecated and I hope to see it
carved out+tagged+deleted. I was primary on godoc while on the OSP team
and while working on pkgsite that replaced it, but that is long since
complete.
Continue to de-emphasize my x/website involvement as started in
CL 335209 since my team isn't the primary owner of x/website by now.
Change-Id: Idb97d2d3a6486468809522d329e46bf1872da347
Reviewed-on: https://go-review.googlesource.com/c/build/+/457339
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Brad told me he's okay with being removed here. Thanks for all the work!
Change-Id: I3e101a2b612166761e5ef478bb94373513c6cc32
Reviewed-on: https://go-review.googlesource.com/c/build/+/457338
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Start using the latest stable version of Go.
While here, show 8080 as an example local development port in the README
since that's more common in x/build.
Change-Id: Ib2fa17cde2ee4c658dc844c779d308656fdc10cb
Reviewed-on: https://go-review.googlesource.com/c/build/+/457337
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
CL 454935 broke the Kubernetes ingress by requiring IAP on health
checks. Move /healthz handling into internal/https, where it
automatically bypasses authentication and removes some duplicate trivial
implementations.
Unfortunately, GKE is not capable of inferring health check parameters
from a multi-container pod like relui, so we have to change our
BackendConfig. That sets off a yak shave -- I made the questionable
decision to use the same backend for all our IAP services, and the
coordinator doesn't currently support /healthz. Split all them up and
delete the devapp configuration I was using for testing way back in the
day.
Change-Id: I45e866d30508a07e9a805de70af731dd64c22d7f
Reviewed-on: https://go-review.googlesource.com/c/build/+/455215
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Heschi Kreinick <heschi@google.com>
Run-TryBot: Heschi Kreinick <heschi@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Previously for all packages that had {filippo, securityTeam}, only
filippo would get added as a reviewer, because github teams are not
considered when assigning reviewers. In all those cases switch to
{filippo, roland, securityTeam}, so that at least one person on the
Security team is at least assigned as a reviewer.
Additionally remove agl from secondary on various packages, as he hasn't
worked on the project actively in quite a while.
Change-Id: I96fc27b81b7fe54b990948854a4c7140b9c66d32
Reviewed-on: https://go-review.googlesource.com/c/build/+/444055
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Filippo Valsorda <filippo@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Auto-Submit: Roland Shoemaker <roland@golang.org>
Run-TryBot: Roland Shoemaker <roland@golang.org>
If I need to find someone to ping to triage a bug, I look here.
So, I added the OS (and architecture) teams.
Updates #28596.
Change-Id: I32a8eff23d68d6d9219ec71a509a6693963c8f05
Reviewed-on: https://go-review.googlesource.com/c/build/+/443435
Run-TryBot: David Chase <drchase@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
We will move the code review responsibility to persons on rotation.
And update adonovan's gh handle
Change-Id: Idc4ba5eea82cd1616d4c7054ea0d92138082f61a
Reviewed-on: https://go-review.googlesource.com/c/build/+/440335
Reviewed-by: Robert Findley <rfindley@google.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
Add entries for various new code-coverage related packages, and update
some existing packages (ex: cmd/cover) with revised owner info.
Updates golang/go#51430.
Change-Id: Ibf9e040c8a018b04974c78deb8c8e3355f8bec91
Reviewed-on: https://go-review.googlesource.com/c/build/+/436784
Run-TryBot: Than McIntosh <thanm@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
The bug we were working around is fixed in Kubernetes 1.22, which we
have finally upgraded to.
Change-Id: Ic51a80521d72cb81ceb125586632e1101da795aa
Reviewed-on: https://go-review.googlesource.com/c/build/+/422594
Run-TryBot: Heschi Kreinick <heschi@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Jenny Rakoczy <jenny@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
This change will cause a number of compiler-related issues to
automatically gain the compiler/runtime label on GitHub, and helps
distribute the responsibility of ownership.
It also adds a ton of previously unlisted packages now that we can
assign group ownership.
Change-Id: I6ac932b0950cc386326eb3e1b979d5cbdc8a98a0
Reviewed-on: https://go-review.googlesource.com/c/build/+/416416
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
This change moves the devapp deployment documentation into the doc
directory because it's common to all services that run in the GKE
cluster.
It also restructures that documentation just a little to alleviate some
confusion (i.e. make push-prod is not necessary if using make
deploy-prod).
Then, it updates all the relevant services' documentation to refer to
that shared document.
Also, it touch up a couple of the services' READMEs, like gopherbot and
pubsubhelper.
Change-Id: I3774385f1d0ba9e9886ece8462294c1f28e8bd17
Reviewed-on: https://go-review.googlesource.com/c/build/+/416220
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
The README suggests a command to run coordinator locally in dev mode.
Update it to take into account that coordinator uses internal/https for
HTTP(S) serving now and no longer always serves https://localhost:8119.
While here, document that the -env flag isn't needed when -mode=dev is
being used. Fix and simplify the /try-dev page: it was using dev build
constraint, so not tested via CI and broke. (I considered just deleting
it, but it was simple enough to keep and it adds value to dev mode.)
Change-Id: I4ffaecd07e5f1321f55bc18f42281d48f2433f30
Reviewed-on: https://go-review.googlesource.com/c/build/+/416215
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Alex Rakoczy <jenny@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
As mentioned in GKE docs¹, the default is 30 seconds, which is too short
for the /logs endpoint to serve a 304 while long-polling during periods
of low corpus activity. Use a minute, since that worked okay before.
While here, update other YAML files to use the non-beta annotation for
the backend config, since we're using a new enough Kubernetes by now.
¹ https://cloud.google.com/kubernetes-engine/docs/how-to/ingress-features#timeoutFixesgolang/go#53569.
Change-Id: If35ac2b30a8fcb1740d0597afd3e4cda4fe0cea1
Reviewed-on: https://go-review.googlesource.com/c/build/+/415354
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
This change adds a task to gopherbot to automatically label
compiler/runtime issues based on the owners list. This task will aid in
finding compiler/runtime issues for further triaging, prioritization,
and assigning ownership.
This change also adds a JSON field to the code owners API to allow for
obtaining all ownership information. The reason why an explicit field
was chosen over alternatives like an empty path set meaning 'all' is
because it's backwards compatible.
Finally, it exports the translation that the owners page does for
package names so that gopherbot can translate the paths the owners
page returns into the names used for packages in GitHub issues.
Change-Id: Idb81a2ab53d16f8bec71450899ea33a5e1c30abe
Reviewed-on: https://go-review.googlesource.com/c/build/+/413579
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
This change adds license headers to files which are missing them.
A modified version of the addlicense tool was used to add these headers:
https://github.com/google/addlicense
The command issued was:
addlicense -c "Go Authors" -l bsd *
Change-Id: Id6fd17c587f99b3ff0ef5f5f35befcfe5aaeef72
Reviewed-on: https://go-review.googlesource.com/c/build/+/409614
Run-TryBot: Carlos Amedee <carlos@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Alex Rakoczy <alex@golang.org>
CL 407394 added a detailed error in case of a rare, unexpected problem.
Propagate it in one more place so we benefit from the new information
if this happens again.
Updates golang/go#47695.
Updates golang/go#50474.
Change-Id: Iddb9a8d8de320d535fd15842c60bf0274391f8f2
Reviewed-on: https://go-review.googlesource.com/c/build/+/408194
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
Hana (Hyang-Ah) Kim
Zxilly
Dmitri Shuralyov
Michael Pratt
Russ Cox
Suzy Mueller
Than McIntosh
Bryan C. Mills
Ian Lance Taylor
Rob Findley
Mauri de Souza Meneguzzo
Heschi Kreinick
Carlos Amedee
M Hickford
Filippo Valsorda
Michael Anthony Knyszek
cui fliter
Roberto Clapis
Roland Shoemaker
David Chase
Tim King