We're still working on this on our side, and will update this after it's resolved or more progress is made. Thanks for your patience and for pushing this forward.
Looking more, I found we removed these from v3.3 in #356 (to fix #351). So I agree it seems this needs more investigation. Also see past discussion in https://github.com/glfw/glfw/issues/2133 and htt…
@joedian It was a draft CL associated with a review comment. Russ has [go.dev/cl/575255](https://go.dev/cl/575255) which is still open but needs attention.
This is similar to (or the same as) #12822 and likely some more gofmt issues. Also consider this instance:
```Go
switch {
case true:
// comment 1
// comment 2
// comment 3
// comment 4
/…
Seems okay to me.
We had this in v3.2 but didn't propagate it forward to v3.3:
https://github.com/go-gl/glfw/blob/a69d953ea14231b2f19998f0b434b6c1e6c6f710/v3.2/glfw/build.go#L25-L26
(It was …
This still happens with Build 2096 for me.
I also noticed something strange: this happens reproducibly as soon as I change safe.bareRepository to explicit on my primary macOS machine (so I'm unabl…
(2 comments)
The 'Minor changes to the library' section is intended for changes to existing packages. Something like a new package being added should go under 'Core library' section, similar to `doc…
From what I can tell, SWARMING_NEVER_REBOOT has effect for most frequent reasons that would otherwise cause the reboot to happen, but it doesn't catch all. The swarming bot seems to occasionally trig…
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…
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 …
[CL 550215](https://go.dev/cl/550215) added a new package. One of its tests, `TestPQCrystalsAccumulated`, is failing on the linux/s390x port:
```
--- FAIL: TestPQCrystalsAccumulated (0.07s)
…
@panjf2000 The Auto-Submit+1 vote is documented at https://go.dev/wiki/GerritAccess#auto-submit. I meant you can ask the person who left the +2 vote to either submit or leave Auto-Submit+1 vote—but…
Hi @panjf2000, thanks for contributing to the Go project and asking for approvers access again.
We considered this request. As documented at https://go.dev/wiki/GerritAccess, Gerrit "approvers" ac…
Hi @qiulaidongfeng, thanks for your interest and contributions to Go.
We considered this request. Similarly to https://github.com/golang/go/issues/66611#issuecomment-2052070282, when considering g…
Hi @mauri870, thanks for contributing to the Go project and asking for this access.
We considered this request and felt your contribution history demonstrates enough signals for us to be close to …
Hi @qiulaidongfeng, thanks for your interest and contributions to Go.
We considered this request (previously considered in #64255). As documented at https://go.dev/wiki/GerritAccess, Gerrit "appro…
Hi @raghvenders, thanks for your contributions to Go so far.
We discussed this request. We generally look for contributors to have a more established track record of contributions before consideri…
This is resolved as of [CL 569875](https://go.dev/cl/569875). (It didn't get closed automatically because the Fixes line didn't specify the `golang/go` repo quite right.)
@jub0bs Yes, your understanding is right and generally it's good to allow some time for discussion on the issue first, though it's still okay to send a change sooner. There are some more details abou…
Thanks. Confirming a few details.
`GOARCH` should be `wasm32` instead of `wasm`, is that right?
Related to the above—will `GOROOT/misc/wasm` be used for both `wasm` and `wasm32` exec helpers? Tha…
Hi abner chenc,
Thanks for your contributions and helping maintain the linux/loong64 port. We've added you to the `project-golang-may-start-trybots` group, so you should be able to trigger [trybot…
> should backport the fix as well
To clarify, do you think the fix to be backported should include changes in [CL 576756](https://go.dev/cl/576756) and [CL 576895](https://go.dev/cl/576895)?
Here's the certificate: [linux-ppc64-power8-1712341779.cert.txt](https://github.com/golang/go/files/14889449/linux-ppc64-power8-1712341779.cert.txt).
I'll also work on another CL needed for that h…
The `pollLoop` method that runs in a loop usually takes a minute, but sometimes seems to get stuck and eventually completes in just over 24 hours without any error. This is a tracking issue to sort t…
The definition/embed.txt, definition/misc.txt, and hover/sizeoffset.txt cases of `Test` are failing on linux/arm (32-bit) since they were introduced in [CL 573076](https://go.dev/cl/573076).
For e…
Thanks for reporting.
As a side note, the [package comment](https://pkg.go.dev/log/syslog#pkg-overview) states the `log/syslog` package is frozen and that some external packages provide more funct…
Here's the certificate for the bot hostname `linux-ppc64le-power10`:
[linux-ppc64le-power10-1712180608.cert.txt](https://github.com/golang/go/files/14858800/linux-ppc64le-power10-1712180608.cert.t…
Thanks. Here's the certificate: [linux-ppc64le-power9-1712180354.cert.txt](https://github.com/golang/go/files/14858793/linux-ppc64le-power9-1712180354.cert.txt).
I've initially observed this on various builders, most often linux-386 ones, but it seems to reproduce on `darwin/arm64` too.
At tip without x/net@v0.23.0 (e.g., commit 61a3ee54411086d4bf996b65dd3…
Thanks for the ping @pmur. I have time to help move this forward this week.
I've mailed [CL 575856](https://go.dev/cl/575856) as the next step here, which should make it possible for you to start …
This seems to also happen at tip (commit 94dba61276f26c01b7d3d1a4abb195bdcc596bfd) if attempting to make.bash with `GODEBUG=gotypesalias=1`.
<details>
```
src $ git pull --ff-only
Already up…
The Windows installer currently always takes responsibility for setting the `GOPATH` environment variable (to `%USERPROFILE%\go`) and adding `%GOPATH%/bin` to `PATH`. If a user wishes to use a differ…
Thanks for reporting. https://pkg.go.dev/reflect#StructTag documentation says:
> By convention, tag strings are a concatenation of optionally space-separated key:"value" pairs. Each key is a non-e…
https://go.dev/doc/go1.22#os/exec includes:
> On Windows, `Command` and `Cmd.Start` no longer call `LookPath` if the path to the executable is already absolute and has an executable file extension…
> which would imply a builder with at least least Clang 15
There's a LUCI builder with clang 15 (e.g., [gotip-linux-amd64-clang15](https://ci.chromium.org/p/golang/builders/ci/gotip-linux-amd64-cl…
> you have to do [...] `DIR=$(mktemp -d); cd $DIR; go version`
If you'd like to determine the bundled Go toolchain's version, it's possible to do it with `GOTOOLCHAIN=local go version`. This is do…
It's common for Go tests that require certain capability or dependency to skip themselves if the environment doesn't meet that requirement. For example, see the various `MustHave*` helpers in [`inter…
Let's consider starting to do that in the new-to-be v3.4/glfw module (#393). What would some of the finer details look like? I wrote the following to try to answer that and explore options.
Its fi…
v3.3/glfw is its own module, separate from the top-level module that has the older versions v3.{0,1,2}/glfw. That seems to be working well, so I expect we can repeat that pattern and also have v3.4/g…
I can confirm the same program no longer panics now that GLFW 3.3.10 is pulled in. `glfw.GetKeyName(glfw.KeyUnknown, 0x003f)` returns an empty string. I think we can close this unless there's somethi…
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 …
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…