Activity
Last Week
dmitshur reviewed +2 on golang.org/x/build/main.star: add a goamd64v4 builder2d
Thanks.dmitshur reviewed +2 on golang.org/x/build/main.star: add noavx512 builders running on n2d-standard-16 machines2d
Thanks.dmitshur reviewed +2 on cmd/dist: drop sharding for testdir tests2d
Thanks.
This variant name is the same as the package name, which doesn't convey much information about how this particular test variant differs from the nominal "go test cmd/internal/testdir" invoca…dmitshur commented on cmd/dist: run testdir tests in initial go test run2d
(1 comment)
(Sorry, I meant to post this comment in CL 853365 but accidentally posted it the wrong CL. Feel free to apply it anywhere, or even better all this "drop unused sharding" functionality fe…dmitshur commented on cmd/dist: run testdir tests in initial go test run2d
(1 comment)
Remove these since you're removing the only reason these still exist?dmitshur commented on cmd/dist: run testdir tests in initial go test run2d
(1 comment)
Oh, I see how I misread it - this continues to skip this test on mobile platforms; the reason this special registerTest call is still left behind here is for Wasm targets. That's unfortu…dmitshur commented on cmd/dist: run testdir tests in initial go test run2d
(2 comments)
Consider upstreaming this skip into cmd/internal/testdir itself by adding a `//go:build !race` constraint (similar to [here](https://cs.opensource.google/go/go/+/master:src/sync/pool_te…dmitshur reviewed +2 on cmd/dist: consolidate maxbg logic2d
dmitshur commented on cmd/dist: drop sharding for testdir tests2d
(5 comments)
This CL is a bit hard to review because it removes partial support for sharding cmd/internal/testdir and leaves some unneeded parts of the mechanism behind. It'd be easier to review the…dmitshur reviewed +2 on cmd/dist: consolidate maxbg logic2d
LGTM % remove accidental binary. Thanks.dmitshur commented on cmd/dist: consolidate maxbg logic2d
(1 comment)
Please do not check in an unneeded 4.6 MB binary file. Also added a Hold+1 for that.dmitshur reviewed +2 on golang.org/x/build/env: install podman on Debian 12 images2d
Thanks.dmitshur reviewed +2 on golang.org/x/build/env: allow using podman instead of docker to build images2d
Thanks.
Have you also considered migrating to using Cloud Build to build images? That should be more predictable and requires less local tools installed; we already switched to using Cloud Build for…dmitshur reviewed +2 on golang.org/x/build/main.star: remove freebsd 15.0 builder from known issue list2d
Thanks.dmitshur reviewed +2 on golang.org/x/build/main.star: remove avx512 builders2d
Thanks.dmitshur reviewed +2 on golang.org/x/build/Revert "main.star: wait for capacity for docker instances"2d
Thanks.
```suggestion
This reverts CL 852985.
```
(As suggested at https://go.dev/wiki/CommitMessage#reverts.)dmitshur commented on golang.org/x/build/main.star: wait for capacity for docker instances3d
(1 comment)
Also, expiration_timeout for linux-amd64_docker can probably stay at 6 hours (for the same reason it's 6 hrs for all other builders on GCE). But it's not a big deal either way in the sho…dmitshur reviewed +2 on golang.org/x/build/main.star: wait for capacity for docker instances3d
Thanks.
To comment to this part of the commit message:
> (I'm not sure why we don't always wait for capacity in case the builder isn't available).
This part of the comment motivates why.
If we fi…dmitshur commented on golang.org/x/build: upgrade freebsd builders to 14.5 and 15.13d
CC @golang/freebsd, @golang/release.dmitshur commented on golang.org/x/build: add OpenBSD 8.0 builder3d
CC @golang/openbsd, @golang/release.dmitshur fixed an issue golang.org/x/build/internal/installer/darwinpkg: unclear purpose and effect of "Fixing permissions" step during postinstall5d
The "Fixing permissions" step removal in Go 1.27 didn't seem to uncover anything unexpected, so closing this as done.
<details><summary>Confirmation of removal details</summary>
```shell
$ pkgutil …dmitshur fixed an issue golang.org/x/build/internal/installer/darwinpkg: unclear purpose and effect of "Fixing permissions" step during postinstall5d
The "Fixing permissions" step removal in Go 1.27 didn't seem to uncover anything unexpected, so closing this as done.
<details><summary>Confirmation of removal details</summary>
```shell
$ pkgutil …dmitshur commented on golang.org/x/build: add GOAMD64=v3 perf builders5d
There were GOAMD64=v3 perf builders added above, and they're running. It's also added to the perf dashboard.
@matloob Should this be closed as done or was there something specific still left to do h…dmitshur pushed to
master in github.com/mdempsky/unconvert5ddmitshur merged a change github.com/mdempsky/unconvert/...: update go and dependency versions- #705d
dmitshur reviewed +2 on github.com/mdempsky/unconvert/...: update go and dependency versions- #705d
Thanks.
Note that in the general case, it's not necessary to include the -u flag in the "go get" invocation in order to update dependencies. The -u flag means to update indirect dependencies (depend…dmitshur fixed an issue golang.org/x/build: linux-amd64-race is taking about 5 minutes too long for pre-submit purposes6d
dmitshur commented on golang.org/x/build: linux-amd64-race is taking about 5 minutes too long for pre-submit purposes6d
linux-amd64-race trybots are now consistently averaging 5-6 minute completion time (with around 7 minutes in the worst case) after @matloob's work in #82017. That's well within 15 minutes. Calling th…Earlier
dmitshur commented on cmd/compile: ICE "Int64Val(uint64(9223372036854775808))" when ranging over an unsigned constant ≥ 2^63 in a clear loop (`for i := range uint64(1<<63) { s[i] = 0 }`)1w
CC @golang/compiler.dmitshur opened an issue golang.org/x/build: frequent builder infra failures, including linux/amd64 and linux/arm64, on 2026-10-151w
We're aware of an issue most visible on the [Go tip post-submit dashboard](https://ci.chromium.org/p/golang/g/go-gotip/console), where many of the linux/amd64 and linux/arm64 builds are frequently ru…dmitshur commented on golang.org/x/build/cmd/golangbuild: add support for test artifacts in LUCI1w
The latest status here is that functionality is enabled in x/pkgsite, for docker builder. What's left is enabling it for the remaining Go repositories and probably posting something in golang-dev so …dmitshur starred github.com/qmuntal/gobuilder1w
dmitshur commented on cmd/link/internal/ppc64: genpltstub panics instead of reporting a link error when the caller symbol has no data2w
CC @golang/ppc64.dmitshur commented on dmitri.shuralyov.com/website/gido: Support proposals1mo
Proposals are already supported, in the sense they're listed as issues at https://goissues.org/ with a "Proposal" label.
See [prefixtitle.ParseIssue](https://pkg.go.dev/dmitri.shuralyov.com/go/prefi…dmitshur pushed to
main in dmitri.shuralyov.com/service/change1ydmitshur commented on dmitri.shuralyov.com/blog: Trying out the Oculus Rift CV1 for the first time2y
July 4, 2024. Today I got a chance to try out the Apple Vision Pro. :)dmitshur pushed to
main in dmitri.shuralyov.com/scratch2ydmitshur created package in dmitri.shuralyov.com/scratch/quine2y
dmitshur merged a change dmitri.shuralyov.com/scratch: quine: add package quine2y
dmitshur reviewed +2 on dmitri.shuralyov.com/scratch: quine: add package quine2y
LGTM. Thanks, and congrats on your first new Go package here!
Done in PS 2.
Patch set 2 seems to resolve this.dmitshur merged a change dmitri.shuralyov.com/scratch: add AUTHORS.md file2y
dmitshur reviewed +2 on dmitri.shuralyov.com/scratch: add AUTHORS.md file2y
Thanks! If it's ready to be submitted from your side, I can do that now.dmitshur pushed to
main in dmitri.shuralyov.com/html/belt2ydmitshur commented on dmitri.shuralyov.com/scratch: main: add package quine2y
They are. Although Go can make them too easy via `go:embed`.
See some initial comments.
I suggest either adding a quite that can be licensed via the existing BSD-3-Clause LICENSE at the top level o…dmitshur reviewed +2 on dmitri.shuralyov.com/scratch: add AUTHORS.md file2y
Thanks! Congrats on your first change. :D
See one inline comment that's unresolved (yellow border). You can leave a draft comment like "Done." and mark the 'Resolved' checkbox, then when you upload …dmitshur pushed to
main in dmitri.shuralyov.com/go/generated3ydmitshur pushed to
main in dmitri.shuralyov.com/gpu/mtl3ydmitshur pushed to
main in dmitri.shuralyov.com/gpu/mtl3ydmitshur commented on dmitri.shuralyov.com/idiomatic-go: Use defer if its overhead is relatively negligible, or it buys meaningful readability4y
I agree with the points made above. This entry tried to say "use this language feature when it's appropriate; don't use it when it's not", which can be said for any language feature. I'll close this …dmitshur pushed to
master in dmitri.shuralyov.com/service/change4ydmitshur closed an issue dmitri.shuralyov.com/go/generated: update implementation for 2020 spec changes4y
dmitshur merged a change dmitri.shuralyov.com/go/generated: update implementation for 2020 spec changes4y
dmitshur commented on dmitri.shuralyov.com/go/generated: update implementation for 2020 spec changes4y
Go fuzzing support is out of beta and no longer uses this build constraint. It should be replaced with `go1.18` by now.
Done in PS 3.
I'm following the Go style for commit messages, which chooses…dmitshur commented on dmitri.shuralyov.com/go/generated: update implementation for 2020 spec changes4y
Shortened 'golang.org' to 'go.dev' here, and on line 24.dmitshur pushed to
master in dmitri.shuralyov.com/website/gido5ydmitshur opened an issue dmitri.shuralyov.com/go/generated: update implementation for 2020 spec changes5y
dmitshur pushed to
master in dmitri.shuralyov.com/gpu/mtl5ydmitshur pushed to
master in dmitri.shuralyov.com/website/gido6ydmitshur pushed to
master in dmitri.shuralyov.com/website/gido6ydmitshur pushed to
master in dmitri.shuralyov.com/html/belt6ydmitshur closed an issue github.com/shurcooL/issuesapp: Improve use of http.Error.6y
dmitshur commented on github.com/shurcooL/issuesapp: Improve use of http.Error.6y
Resolved a while ago in [commit `431bb22d`](https://github.com/shurcooL/issuesapp/commit/431bb22d86ad0a95db0780e27ab6221b49238464).dmitshur commented on dmitri.shuralyov.com/blog: Replacing WordPress with GitHub Issues-like blog comments6y
I've decided to publish this post on the [blog front page](https://dmitri.shuralyov.com/blog) now, so re-opening.dmitshur reopened an issue dmitri.shuralyov.com/blog: Replacing WordPress with GitHub Issues-like blog comments6y
dmitshur opened an issue dmitri.shuralyov.com/blog: A half-hour to learn Rust, except it's Go instead of Rust6y
dmitshur pushed to
master in dmitri.shuralyov.com/website/gido6ydmitshur pushed to
master in dmitri.shuralyov.com/go/prefixtitle6ydmitshur closed an issue dmitri.shuralyov.com/website/gido: matching by import path pattern misses many packages6y
dmitshur commented on dmitri.shuralyov.com/website/gido: matching by import path pattern misses many packages6y
This is a valid issue, but the root cause analysis was incorrect.
> This happens because they're added using the original paths, without checking that those paths correspond to any existing director…dmitshur pushed to
master in dmitri.shuralyov.com/website/gido6ydmitshur commented on dmitri.shuralyov.com/website/gido: dynamically maintain list of packages (directories)6y
Another place that can be made better is the list of release branches in `initExistingDirectories`. It's currently hard-coded, but should be automated:
```Go
func initExistingDirectories() error {
…dmitshur pushed to
master in dmitri.shuralyov.com/app/changes6ydmitshur created package in dmitri.shuralyov.com/gpu/mtl/example/movingtriangle/internal/coreanim6y
Package coreanim provides access to Apple's Core Animation API (https://developer.apple.com/documentation/quartzcore).
dmitshur created package in dmitri.shuralyov.com/gpu/mtl/example/movingtriangle/internal/appkit6y
Package appkit provides access to Apple's AppKit API (https://developer.apple.com/documentation/appkit).
dmitshur pushed to
master in dmitri.shuralyov.com/gpu/mtl6ydmitshur pushed to
master in dmitri.shuralyov.com/gpu/mtl7ydmitshur pushed to
master in dmitri.shuralyov.com/service/change7ydmitshur pushed to
master in dmitri.shuralyov.com/text/kebabcase7ydmitshur pushed to
master in dmitri.shuralyov.com/route/gerrit7ydmitshur pushed to
master in dmitri.shuralyov.com/state7ydmitshur pushed to
master in dmitri.shuralyov.com/scratch7ydmitshur pushed to
master in dmitri.shuralyov.com/app/changes7ydmitshur pushed to
master in dmitri.shuralyov.com/route/github7ydmitshur pushed to
master in dmitri.shuralyov.com/html/belt7ydmitshur pushed to
master in dmitri.shuralyov.com/route/github7ydmitshur pushed to
master in dmitri.shuralyov.com/service/change7ydmitshur commented on dmitri.shuralyov.com/app/changes: https fetch: Get https://dmitri.shuralyov.com/app/changes?go-get=1: x509: certificate signed by unknown authority7y
I’m glad that it helped. Don’t hesitate to let me know if the issue happens again.dmitshur commented on dmitri.shuralyov.com/app/changes: https fetch: Get https://dmitri.shuralyov.com/app/changes?go-get=1: x509: certificate signed by unknown authority7y
Hello, thanks for the report.
Is this problem reproducible for you? If so, can you share commands for me to run so I can try to reproduce it too?
For question 1, this is the canonical source, but y…dmitshur opened an issue dmitri.shuralyov.com/website/gido: matching by import path pattern misses many packages7y
dmitshur closed an issue dmitri.shuralyov.com/website/gido: same issue or CL may appear multiple times for pattern queries7y
dmitshur pushed to
master in dmitri.shuralyov.com/website/gido7ydmitshur pushed to
master in dmitri.shuralyov.com/app/changes7ydmitshur opened an issue dmitri.shuralyov.com/website/gido: same issue or CL may appear multiple times for pattern queries7y
dmitshur commented on dmitri.shuralyov.com/website/gido: dynamically maintain list of packages (directories)7y
Commit [5275a64a](https://dmitri.shuralyov.com/website/gido$commit/5275a64acd14885a8d67315e2b66570347b757af) has made this better. Directories are now determined by walking Go repositories at process…dmitshur pushed to
master in dmitri.shuralyov.com/app/changes7ydmitshur pushed to
master in dmitri.shuralyov.com/service/change7ydmitshur pushed to
master in dmitri.shuralyov.com/website/gido7y