Today
Yesterday
dmitshur starred github.com/rogpeppe/gohack23h
dmitshur commented on crypto/x509: macOS -framework Secur… in github.com/golang/go1d
As another data point, I am not seeing any ld warnings on the following configuration: OS: macOS 10.13.6 Go version: current tip-1 (c6118af55864916c2c3b1bca8e216e627f232bf3) Xcode: not installed…
dmitshur commented on cmd/go: error when enable cgo and s… in github.com/golang/go1d
> But your suggestion is good, it may be the go-ethereum's problem. > I will report an issue to their project. Sounds good. Feel free to link it to this issue so we can keep track of progress. …
dmitshur commented on x/mobile/cmd/gomobile: c++ with SWIG in github.com/golang/go1d
This issue was resolved by [CL 134975](https://golang.org/cl/134975), but didn't get closed automatically because the commit message wasn't using the pattern described at https://golang.org/wiki/Comm…
This Week
dmitshur commented on Make the mod time optional in github.com/shurcooL/vfsgen1d
Hi @jribe, thanks for the PR! However, it is already possible to make the modify times zero, and it can be done without making direct changes to vfsgen. The way it can be done is with a wrapper `h…
dmitshur commented on x/net/html: html.Parse() panic caus… in github.com/golang/go1d
Great! We can close this issue then, if there's nothing more to do here.
dmitshur commented on Include response body in 'unexpecte… in github.com/shurcooL/graphql2d
Thanks for the issue report @cjwagner, and for providing the details. This issue is already on my radar, as PR #24 was sent to address a similar need. See discussion there for some background and …
dmitshur commented on v2: Add go.mod in github.com/russross/blackfriday2d
I'm not sure that the `/v2` suffix belongs here. I think it's expected to come from the tag version (e.g., tag [v2.0.0](https://github.com/russross/blackfriday/releases/tag/v2.0.0) would have the mod…
dmitshur commented on v2: Add go.mod in github.com/russross/blackfriday2d
It seems that we need to ensure that the installation instructions work for GOPATH workspaces and in module-aware mode, but they don't necessarily have to be the same. For instance, for GOPATH, it ca…
dmitshur commented on vfsgendev: look up source in the cu… in github.com/shurcooL/vfsgen2d
Thanks for the PR. When running inside a Go module, would you expect `vfsgendev` to look for packages only within said module? As far as I know, the usual approach to adding support for modules…
dmitshur commented on runtime: cgo Unix C code can't modi… in github.com/golang/go2d
This issue has a high similarity to #22302 in nature, which was resolved with [CL 75751](https://golang.org/cl/75751). Perhaps that is a good example to follow? It stands out that unlike `runtime.…
dmitshur commented on x/build/cmd/gerritbot: deployed ver… in github.com/golang/go2d
I see a bug in maintner code. It's likely the bug causing this crash, but I don't have full information to say that conclusively. I'll start by fixing it and see if anything more needs to be done.…
dmitshur commented on x/build/cmd/gerritbot: deployed ver… in github.com/golang/go2d
Seeing a few more instances of the same crash. This time it should be easier to debug because the deployed version is known/latest: ``` 2018/09/15 16:16:52 Reloaded data from log *maintner.netMut…
dmitshur commented on x/tools/godoc/vfs: path separator c… in github.com/golang/go2d
Actually, in the [`vfs.OS`](https://godoc.org/golang.org/x/tools/godoc/vfs#OS) documentation, it says: > ... because the slash-separated path passed to Open ... Which implies that the paths are…
dmitshur commented on proposal: x/tools/cmd/godoc: add "j… in github.com/golang/go2d
Here's a 24-second video to demonstrate the workflow and navigation speed possible with the "jump to identifier" dialog. In the video, I open the docs for `net/http` package and visit, in order, the …
dmitshur commented on x/tools/cmd/godoc: add hamburger bu… in github.com/golang/go2d
> Currently, when going through larger sets of documentation, it can be a little annoying to scroll down to a method, only to have to scroll back to the top and repeat. For libs/projects that don't h…
dmitshur commented on x/tools/cmd/goimports: Allow "-loca… in github.com/golang/go2d
Related issue is #20818.
dmitshur commented on cmd/go: error when enable cgo and s… in github.com/golang/go2d
You're saying that `go build --ldflags "-linkmode external -extldflags -static" -a -o main -v -mod=vendor secp256k1.test.go` produces an error above, but there's no error if you take out `-mod=vendor…
dmitshur commented on x/net/html: html.Parse() panic caus… in github.com/golang/go2d
I can't reproduce this with the current `golang.org/x/net/html`. Is this still an open issue? Perhaps it got resolved during changes for #23071, likely [CL 123776](https://golang.org/cl/123776).
dmitshur commented on Inferred package links not always c… in github.com/golang/gddo2d
/cc @dsnet FYI. This issue is something to be aware of when trying to linkify identifiers within the docs.
dmitshur commented on cmd/vet: check for http.ResponseWri… in github.com/golang/go2d
/cc @alandonovan @josharian @mvdan per [owners](https://dev.golang.org/owners/).
dmitshur commented on x/mobile: The generated jni functio… in github.com/golang/go2d
> ### Does this issue reproduce with the latest release? > > The gomobile version is the latest That question is referring to the version of Go you have installed. The latest release is Go 1.11.…
dmitshur commented on x/mobile/cmd/gobind: non-pointer st… in github.com/golang/go2d
/cc @eliasnaur @hyangah per [owners](https://dev.golang.org/owners/).
dmitshur deleted branch in github.com/shurcooL/vfsgen2d
README-usage
dmitshur pushed to master in github.com/shurcooL/vfsgen2d
33ae1944be3fe078a54c597f107e0867da19c713README: Expand the usage section.
dmitshur commented on Port to WASM in github.com/gopherjs/websocket3d
This was resolved in bc9bf10c75f863bd82ad9e5bc2a2c832293383c6.
dmitshur commented on Port to WASM in github.com/gopherjs/websocket3d
The `dataArray := js.TypedArrayOf(data)` line below needs to be updated, doesn't it?
dmitshur commented on 8c859bce4beadf04ba687d0efef6619ce2abca2amodules: use stable tag… in github.com/google/go-github3d
When I do `go mod tidy`, it actually removes those lines, and makes some other changes. How did you update these files @willnorris?
dmitshur commented on x/mobile: Replace hardcoded -miphon… in github.com/golang/go3d
Fixed by [CL 135415](https://golang.org/cl/135415), closing. (It didn't get the "Fixes" format quite right, hence didn't get closed automatically.)
dmitshur commented on Add StartWithSize to allow resizing… in github.com/kr/pty3d
Nitpick, this sentence is missing a period at the end.
dmitshur commented on 8c859bce4beadf04ba687d0efef6619ce2abca2amodules: use stable tag… in github.com/google/go-github3d
Is it expected for `github.com/google/go-querystring v0.0.0-...` lines to not go away? Any idea why not?
dmitshur commented on v2: Add go.mod in github.com/russross/blackfriday3d
The canonical import path of blackfriday v2 is `gopkg.in/russross/blackfriday.v2`, per https://github.com/russross/blackfriday#installation. How does this intersect with that?
dmitshur commented on doc: FAQ entry explaining Go is spe… in github.com/golang/go3d
For anyone who needs to link to this in advance of next Go release, it shows up at tip.golang.org already: https://tip.golang.org/doc/faq#go_or_golang
dmitshur commented on Port to WASM in github.com/gopherjs/websocket3d
This is great, thank you for working on this @nightexcessive! I tested this quickly with [one of my projects](https://github.com/shurcooL/eX0/tree/master/eX0-go#readme) that uses WebSockets substanti…
Last Week
dmitshur commented on GOPROXY using file:// protocol not … in github.com/golang/go4d
/cc @rsc @bcmills
dmitshur commented on Minor documentation fix for beginne… in github.com/shurcooL/vfsgen4d
@tariq1890 I tried to elaborate and clarify the Usage section in PR #53. Can you let me know if you think it's helpful?
dmitshur created branch in github.com/shurcooL/vfsgen4d
README-usage
dmitshur commented on Adding gitignore to vfsgen in github.com/shurcooL/vfsgen4d
Also, I'm happy you're learning how to use vfsgen! If you have any further questions about it, feel free to ask.
dmitshur commented on Adding gitignore to vfsgen in github.com/shurcooL/vfsgen4d
Thanks for answering. In general, I prefer to have as few files in repos as possible: only the ones that are needed and provide clear value (e.g., .go code files, README, etc.). Having unused file…
dmitshur commented on Update install.md in github.com/derekparker/delve4d
Isn’t this path missing the `/src/` path component?
dmitshur commented on Adding gitignore to vfsgen in github.com/shurcooL/vfsgen4d
Hi, thanks for the PR. Can you please elaborate on how you think this change is useful?
dmitshur commented on Make TravisCI build green again in github.com/andygrunwald/go-gerrit5d
A trivial thing I just spotted: `body[:]` could be simplified to `body`. It's already a slice, no need to slice it again.
dmitshur commented on WebAssembly port in github.com/robpike/ivy5d
> if I'm not mistaken, the code for [iOS, Android apps] lives in this repository. I was mistaken, it looks like what lives here is a small part of the mobile ports. The actual iOS/Android app proj…
dmitshur commented on x/build: dashboard “first-class p… in github.com/golang/go5d
I can fix the discrepancy, but I'll need the decision of which are to be considered first-class vs not. Here's the current first-class port list from build.golang.org in text form: darwin/386: …
dmitshur commented on cmd/go/internal/modfetch: TestCodeR… in github.com/golang/go5d
It's not urgent to fix this, I just filed the issue so we don't forget. The test was added 3 months ago in [CL 118095](https://golang.org/cl/118095) by @rsc. I'd wait for his comments on how to be…
dmitshur commented on cmd/go/internal/modfetch: TestCodeR… in github.com/golang/go5d
From a glance, it looks like the test expects the tag not to be there.
dmitshur commented on x/tools/cmd/goimports: Allow "-loca… in github.com/golang/go5d
I think it's highly beneficial to avoid adding flags if possible. gofmt is already pretty opinionated, and people accept that. It has no flags (more or less), which means all projects are formatted t…
dmitshur commented on x/build: limit the number of auto-a… in github.com/golang/go5d
I'll wait to hear @andybons thoughts on this, but I'd say if we're going to treat "would ping more than 5 people" specially, it should just avoid doing anything. I don't think it's viable to pick …
dmitshur opened an issue in github.com/robpike/ivy5d
dmitshur commented on x/build/cmd/gerritbot: deployed ver… in github.com/golang/go6d
Haven't seen any crashes since then. I'll consider this resolved until there's evidence otherwise (we can re-open this).
dmitshur commented on How I use GOPATH with multiple work… in dmitri.shuralyov.com/blog6d
@hemenkapadia It might be worth asking that question on a [dep forum](https://gophers.slack.com/messages/dep-users). I am not familiar with it, so I can't say. I would expect that it should not fail …
dmitshur commented on Make TravisCI build green again in github.com/andygrunwald/go-gerrit1w
Not at a problem at all, this isn't blocking me in any way. If you don't want this opportunity to improve code to be forgotten, I'd suggest opening an issue to track it.
dmitshur created branch in github.com/shurcooL/ivybrowser1w
wasm
dmitshur created branch in github.com/goxjs/glfw1w
wasm
dmitshur created branch in github.com/goxjs/gl1w
wasm
dmitshur commented on syscall/js: unclear what the zero j… in github.com/golang/go1w
> Open to debate. I have a relevant mini-experience-report to share. In the [WebGL 1 spec](https://www.khronos.org/registry/webgl/specs/latest/1.0/), two of the functions are defined as follows…
dmitshur commented on Add WebAssembly support (GOARCH=was… in github.com/gopherjs/websocket1w
Thanks for opening the issue, and for sending a draft PR! I tested it with one of my projects that uses WebSockets just now, and I can see that it compiles and functions partially, but after some …
dmitshur commented on x/build/internal/gophers: first ema… in github.com/golang/go1w
I understand the issue fully now. I have a fix in mind that I think will be good, and will send a CL (with a commit message that explains everything) soon. I also want to follow up by improving th…
dmitshur commented on access: may-start-trybots in github.com/golang/go1w
Hi Alessandro, Thank you for your contributions, and for requesting trybot access! I've added you to the `may-start-trybots` group, so you should be able to do so now. As you've probably seen a…
dmitshur closed an issue in github.com/golang/go1w
dmitshur commented on x/build/internal/gophers: first ema… in github.com/golang/go1w
> You cannot add a person to a review using their Gerrit ID email, for instance. @andybons Could you please provide more information about that. I'd like to get a better understanding of the compl…
dmitshur commented on Request: use *js.Value instead of j… in github.com/gopherjs/gopherwasm1w
> as the behavior uninitialized `js.Value` isn't clearly (if at all) defined to me. For posterity, that is being tracked in issue golang/go#27592.
dmitshur deleted branch in github.com/dmitshur/dl1w
rm-blank-line
dmitshur commented on Minor documentation fix for beginne… in github.com/shurcooL/vfsgen1w
Thanks for the PR! I'm a bit worried that the very next paragraph talks about `assets`, but it's a different one. It's the `assets` variable in the generated .go file, not the input filesystem `fs…
dmitshur commented on Reports zero candidates for syscall… in github.com/mdempsky/gocode1w
This looks related to #56.
dmitshur commented on x/tools/go/packages: TestJSON may b… in github.com/golang/go1w
It happened again on `darwin-amd64-10_11` builder on go1.10 branch: https://build.golang.org/log/c5f6c6b6e6863169c92564c528adbb0cbc163b23 So it's not just FreeBSD that's affected. Just providin…
dmitshur deleted branch in github.com/dmitshur/dl1w
simplify-versionArchiveURL
dmitshur commented on x/build: mirror the "dl" repo to Gi… in github.com/golang/go1w
It seems that getting TryBots to be functional on the new repo would require additional steps that haven't been done yet. When I tried to run them in [CL 134435](https://golang.org/cl/134435), they f…
dmitshur commented on vanity import paths backed by GitHu… in github.com/golang/gddo1w
The fix is merged and deployed, resolving this issue. Thanks again for the report @myitcv! Please let us know if something else comes up.
dmitshur commented on x/build: mirror the "dl" repo to Gi… in github.com/golang/go1w
> The last thing to confirm after this is that googlebot will apply the "cla: yes"/"cla: no" labels on future PRs. Verified that it works well in https://github.com/golang/dl/pull/2: ![image](h…
dmitshur pushed to rm-blank-line in github.com/dmitshur/dl1w
51dbb14703592a95f599231ff29d8692ceb244d4delete blank line from codereview.cfg
dmitshur commented on x/build/cmd/gerritbot: comments occ… in github.com/golang/go1w
Now that https://github.com/golang/go/issues/27561#issuecomment-420072789 is resolved, at least a part of this issue should be resolved too. Perhaps all of it. If there are further occurrences of …
dmitshur commented on x/build/cmd/gerritbot: first "PR ha… in github.com/golang/go1w
This is deployed. I've confirmed it's working. I created a new PR just now, and the "This PR has been imported to Gerrit for code review" comment was posted: https://github.com/golang/dl/pull/2…
dmitshur created branch in github.com/dmitshur/dl1w
simplify-versionArchiveURL
dmitshur commented on tour: Dead link at that page in github.com/golang/tour1w
This issue should be resolved or at least improved when https://github.com/golang/go/issues/27205 is resolved.
dmitshur commented on vanity import paths backed by GitHu… in github.com/golang/gddo1w
That CL can be tested (temporarily) at https://20180910t145226-dot-godoc-org.appspot.com/. E.g., these work as expected: - https://20180910t145226-dot-godoc-org.appspot.com/go4.org/sort (doesn'…
dmitshur commented on vanity import paths backed by GitHu… in github.com/golang/gddo1w
> Does it make sense to revert the commit that broke this in the first place, for now? In this (rare) case, I've opted not to, because the problem and the path to resolution are well understood. T…
dmitshur commented on Using unexported field in query res… in github.com/shurcooL/githubv41w
If you don’t mind, I’ll reopen this issue and use it to track the fix to the panic when unexported fields are involved. Once resolved, future users won’t run into this panic either.