cmd/go: go get -race can't find "sync/atomic" #20512

starius · 2017-05-27T17:09:26Z

Can be a rebirth of #18486

What version of Go are you using (`go version`)?

go version go1.8.3 linux/amd64

This go version was compiled from source using make.bash.

This issue is not reproduced on the released binaries perhaps because of #18486 (comment)

What operating system and processor architecture are you using (`go env`)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/bnagaev"
GORACE=""
GOROOT="/home/bnagaev/go"
GOTOOLDIR="/home/bnagaev/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build015631019=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?

I run go test -race and got weird errors:

$ go get -race
# sync
/scrubbed/src/sync/cond.go:8: can't find import: "sync/atomic"
# strconv
/scrubbed/src/strconv/atof.go:13: can't find import: "math"
# vendor/golang_org/x/crypto/poly1305
/scrubbed/src/vendor/golang_org/x/crypto/poly1305/poly1305.go:21: can't find import: "crypto/subtle"

I created minimal working example here: https://github.com/starius/racesync

To reproduce build go from source and run go get -race github.com/starius/racesync

The text was updated successfully, but these errors were encountered:

AlekSi · 2017-05-27T17:20:22Z

Try to run the following commands:

go install -v std
go install -v -race std

starius · 2017-05-27T17:33:47Z

After running these commands go get -race succeeded.

starius · 2017-05-27T17:34:14Z

Is it a bug or WAI?

The build is expected to fail. See golang/go#20512

CI failed as expected: https://travis-ci.org/starius/gohere/builds/236737281 https://ci.appveyor.com/project/starius/gohere/build/1.0.35 See golang/go#20512

ianlancetaylor · 2017-05-29T19:07:17Z

In your comment text you wrote go test -race but in your title and example you wrote go get -race. I assume you mean go get -race.

This looks like a bug but it doesn't seem like a very important bug. Why are you writing go get -race?

starius · 2017-05-29T19:24:13Z

@ianlancetaylor
Both go get -race and go test -race fail. Initially I found the bug for go test -race but then I found that it has nothing to do with "test" part and used just go get -race in the minimal example.

See golang/go#20512

ianlancetaylor · 2017-06-02T00:20:09Z

I can recreate this with go get but not with go test. I'm working on a fix for go get. If you can show me an example that fails with go test I would appreciate that.

gopherbot · 2017-06-02T00:40:07Z

CL https://golang.org/cl/44630 mentions this issue.

starius · 2017-06-02T01:11:51Z

If you can show me an example that fails with go test I would appreciate that.

Done:

$ go get -t github.com/starius/racesync
$ go test -race github.com/starius/racesync/...
# net
can't create $WORK/net.a: open $WORK/net.a: no such file or directory

may be a different issue though.

davecheney · 2017-06-02T01:28:01Z

I think that's a different issue. Can you please post the output of go test -x -race github.com/starius/racesync/. <http://github.com/starius/racesync/>..

…

On Fri, Jun 2, 2017 at 11:11 AM, Boris Nagaev ***@***.***> wrote: If you can show me an example that fails with go test I would appreciate that. Done: $ go get -t github.com/starius/racesync $ go test -race github.com/starius/racesync/... # net can't create $WORK/net.a: open $WORK/net.a: no such file or directory may be a different issue though. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#20512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA3wJRS7nhohWCjptdUQJsWt8wEE-ks5r_2FYgaJpZM4NoaRM> .

starius · 2017-06-02T01:36:04Z

Output of go test -x -race github.com/starius/racesync/.

https://gist.github.com/d87ab40a2d7e4a1df72588c96f734606

davecheney · 2017-06-02T01:42:49Z

Looks like that succeed. Did you change anything between your previous post and this one? Your go installation is in /home/bnagaev/.goroot, Do you use a tool to manage this for you, or is it a personal choice? Did you or the tool set GOROOT?

…

On Fri, Jun 2, 2017 at 11:36 AM, Boris Nagaev ***@***.***> wrote: Output of go test -x -race github.com/starius/racesync/. https://gist.github.com/d87ab40a2d7e4a1df72588c96f734606 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#20512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA2Kbb3n7D9QSF_BugmLO_BqLyiBZks5r_2cGgaJpZM4NoaRM> .

starius · 2017-06-02T01:54:29Z

I think it succeed because the target was github.com/starius/racesync/. not github.com/starius/racesync/...

For go test -x -race github.com/starius/racesync/... I have
https://gist.github.com/4c41db39b692b8f50ec8bd0d20910b1b

Did you change anything between your previous post and this one?

I didn't.

Do you use a tool to manage this for you, or is it a personal choice?

Both :-)
I wrote a tool to download, build and install Go: https://github.com/starius/gohere
and used it to install Go to that directory.

I patched the tool to run go install -v -race std but I removed GOROOT/pkg/linux_amd64_race/ before running the commands.

Did you or the tool set GOROOT?

No. I also double checked that the variable GOROOT is not set.

davecheney · 2017-06-02T02:05:05Z

The bug may be in the script you used; by copying the files using shutil.copytree(src, dst) after installation, it may have upset the timestamps.

…

On Fri, Jun 2, 2017 at 11:54 AM, Boris Nagaev ***@***.***> wrote: I think it succeed because the target was github.com/starius/racesync/. not github.com/starius/racesync/... For go test -x -race github.com/starius/racesync/... I have https://gist.github.com/4c41db39b692b8f50ec8bd0d20910b1b Did you change anything between your previous post and this one? I didn't. Do you use a tool to manage this for you, or is it a personal choice? Both :-) I wrote a tool to download, build and install Go: https://github.com/starius/gohere and used it to install Go to that directory. I patched the tool to run go install -v -race std but I removed GOROOT/pkg/linux_amd64_race/ before running the commands. Did you or the tool set GOROOT? No. I also double checked that the variable GOROOT is not set. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#20512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA3io1W9F3kx9NnBFlin1EIZH2wszks5r_2tXgaJpZM4NoaRM> .

starius · 2017-06-02T07:58:29Z

I built Go 1.8.3 manually from source and used it from the directory where make.bash installed binaries itself:

$ cd ~/go/src
$ GOROOT_BOOTSTRAP=/home/bnagaev/.cache/gohere/go1.4.3_bootstrap ./make.bash
$ /home/bnagaev/go/bin/go get -t github.com/starius/racesync/...
$ /home/bnagaev/go/bin/go test -x -race github.com/starius/racesync/...

Error: https://gist.github.com/75bd5f630fabbd3f1eb7e4d3d37a33d1

davecheney · 2017-06-02T08:08:57Z

That's is very strange. What is you gopath variable set to?

…

On Fri, 2 Jun 2017, 17:58 Boris Nagaev ***@***.***> wrote: I built Go 1.8.3 manually from source and used it from the directory where make.bash installed binaries itself: $ cd ~/go/src $ GOROOT_BOOTSTRAP=/home/bnagaev/.cache/gohere/go1.4.3_bootstrap ./make.bash $ /home/bnagaev/go/bin/go get -t github.com/starius/racesync/... $ /home/bnagaev/go/bin/go test -x -race github.com/starius/racesync/... Error: https://gist.github.com/75bd5f630fabbd3f1eb7e4d3d37a33d1 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#20512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA1EujNBlpRzD6zpA7pj4_WAYvZ_Uks5r_8CmgaJpZM4NoaRM> .

starius · 2017-06-02T08:14:41Z

GOPATH=/home/bnagaev

davecheney · 2017-06-02T08:51:02Z

Ok, that's fine. Thanks for confirming.

…

On Fri, 2 Jun 2017, 18:14 Boris Nagaev ***@***.***> wrote: GOPATH=/home/bnagaev — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#20512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA1o1MftQg-tgtbb_dAFmHkS_wE9Aks5r_8RygaJpZM4NoaRM> .

ianlancetaylor · 2017-06-02T18:22:50Z

@starius Please open a new issue for the problem you are seeing with go test -race. That is clearly a different problem from the go get -race problem. I do not understand what you are seeing with go test -race. Thanks.

starius added a commit to starius/gohere that referenced this issue May 27, 2017

test on github.com/starius/racesync

4ea2c10

The build is expected to fail. See golang/go#20512

starius added a commit to starius/gohere that referenced this issue May 27, 2017

build race version of standard library

6525c43

CI failed as expected: https://travis-ci.org/starius/gohere/builds/236737281 https://ci.appveyor.com/project/starius/gohere/build/1.0.35 See golang/go#20512

ianlancetaylor changed the title ~~go get -race can't find "sync/atomic"~~ cmd/go: go get -race can't find "sync/atomic" May 29, 2017

ianlancetaylor added this to the Go1.10 milestone May 29, 2017

starius pushed a commit to starius/config that referenced this issue May 29, 2017

update gohere (fix -race problem)

3390146

See golang/go#20512

starius added a commit to starius/config that referenced this issue May 30, 2017

update gohere (fix -race problem)

b5d9725

See golang/go#20512

starius mentioned this issue Jun 2, 2017

cmd/go: go test -race can't create $WORK/net.a #20560

Closed

rsc mentioned this issue Jun 5, 2017

cmd/compile, cmd/link: support arbitrary import file map configurations #20579

Closed

gopherbot closed this as completed in 8f98931 Jun 13, 2017

golang locked and limited conversation to collaborators Jun 13, 2018

gopherbot added the FrozenDueToAge label Jun 13, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

cmd/go: go get -race can't find "sync/atomic" #20512

cmd/go: go get -race can't find "sync/atomic" #20512

starius commented May 27, 2017

AlekSi commented May 27, 2017

starius commented May 27, 2017

starius commented May 27, 2017

ianlancetaylor commented May 29, 2017

starius commented May 29, 2017

ianlancetaylor commented Jun 2, 2017

gopherbot commented Jun 2, 2017

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

ianlancetaylor commented Jun 2, 2017

cmd/go: go get -race can't find "sync/atomic" #20512

cmd/go: go get -race can't find "sync/atomic" #20512

Comments

starius commented May 27, 2017

What version of Go are you using (go version)?

What operating system and processor architecture are you using (go env)?

What did you do?

AlekSi commented May 27, 2017

starius commented May 27, 2017

starius commented May 27, 2017

ianlancetaylor commented May 29, 2017

starius commented May 29, 2017

ianlancetaylor commented Jun 2, 2017

gopherbot commented Jun 2, 2017

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

starius commented Jun 2, 2017

davecheney commented Jun 2, 2017 via email

ianlancetaylor commented Jun 2, 2017

What version of Go are you using (`go version`)?

What operating system and processor architecture are you using (`go env`)?