cmd/go: Duplicate symbols with more than one main package in -coverpkg #32150

imsodin · 2019-05-20T14:52:30Z

I have been told by @bcmills in #30374 (comment) to create a new issue. This report is essentially the same as #30374 (comment):

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

$ go version
go version go1.12.5 linux/amd64

Does this issue reproduce with the latest release?

Didn't try.

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

go env Output

$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/simon/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/media/ext4_data/Coding/go"
GOPROXY=""
GORACE=""
GOROOT="/media/ext4_data/Linux/source/go"
GOTMPDIR=""
GOTOOLDIR="/media/ext4_data/Linux/source/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/simon/tmp/syncthing/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build248314634=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Running all tests with -coverpkg on all packages and using -coverprofile like this:

git clone https://github.com/syncthing/syncthing.git
cd syncthing
go test -short -coverprofile coverage.out -coverpkg ./cmd/...,./lib/... ./cmd/... ./lib/...

What did you expect to see?

No errors.

What did you see instead?

Errors like previously reported in the linked issue:

2019/05/15 16:02:50 duplicate symbol go.constinfo.main (types 45 and 45) in github.com/syncthing/syncthing/cmd/strelaypoolsrv and /home/simon/.cache/go-build/87/87afda35dba0b8de44e2f9d449eda7ce4caef7ed781612883b4e457eab1ee3d5-d(_go_.o)

The same also happens with just ./.... I cleaned out ~/.cache/go-build for good measure, didn't change anything.

To make sure it isn't some weird local problem of mine I also ran it on Syncthing's CI and reproduced the issue there as well: https://build.syncthing.net/viewLog.html?buildId=39242&buildTypeId=Syncthing_Coverage&tab=buildLog

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

bcmills · 2019-05-20T14:58:51Z

CC @jayconrod

imsodin · 2019-05-21T11:17:52Z

Actually -coverprofile has nothing to do with it, the error is the same regardless of whether that flag is used or not. I don't understand how my issue is different from #30374 (@bcmills).

jayconrod · 2019-05-21T19:53:21Z

@imsodin I was able to reproduce this with 1.12.5 but not with tip. I believe this was fixed in CL 176558 as a fix for #31946. That's a slightly different bug from #30374. I didn't realize it affected 1.12.

I'll need to verify 176558 would actually fix this on 1.12.5. If it does, I'll backport to 1.12. If not, I'll investigate this as a new issue. I'm at a conference right now, so it may take a few days though.

jayconrod · 2019-05-28T22:38:09Z

@imsodin Thanks for reporting this, both here and in #30374. I've confirmed that https://golang.org/cl/176558 fixes the issue when cherry-picked onto 1.12.5.

#32295 will track the cherry-pick onto 1.12.

gopherbot · 2019-06-07T13:57:02Z

Change https://golang.org/cl/179677 mentions this issue: [release-branch.go1.12] cmd/go: force -coverpkg main packages to be built as libraries

…uilt as libraries This fixes TestScript/cover_pkgall_multiple_mains, which started failing after CL 174657. When compiling main packages with coverage instrumentation (e.g., for -coverpkg all), we now pass -p with the full import path instead of '-p main'. This avoids link errors 'duplicate symbol main.main (types 1 and 1)'. Fixes #32295 Updates #31946 Updates #32150 Change-Id: Id147527b1dbdc14bb33ac133c30d50c250b4365c Reviewed-on: https://go-review.googlesource.com/c/go/+/176558 Run-TryBot: Jay Conrod <jayconrod@google.com> Reviewed-by: Cherry Zhang <cherryyz@google.com> Reviewed-by: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> (cherry picked from commit 3b8c804) Reviewed-on: https://go-review.googlesource.com/c/go/+/179677

andreasph · 2019-08-01T12:48:17Z

experimenting this issue on circleci with 1.11.5 and 1.11.12 (not on 1.12.7):

GO111MODULE=on TZ= go test -race -p 1 -tags "system" \
		-coverprofile=%REDACTED%/st_coverage.txt -covermode=atomic -coverpkg=%REDACTED%/... ./...

2019/08/01 12:18:43 duplicate symbol go.constinfo.main (types 44 and 44) in  .%REDACTED%/cmd/a-main-package and $WORK/b423/_pkg_.a(_go_.o)

strangely enough it happened after adding a new main package, but we still had multiple ones before and never encountered the issue.

I'm not sure if 1.11 was also affected and the fix was never backported to 1.11 or it is something else,

do I have to open a new ticket?

bcmills · 2019-08-01T21:50:47Z

@andreasph, the fix was not (and is not planned to be) backported to 1.11. The workaround, if needed, is to upgrade to the latest patch release of 1.12, or wait for 1.13 (scheduled to go out soon) and upgrade to that.

andreasph · 2019-08-02T11:24:28Z

@bcmills fair enough, we will bump to 1.12.7. just one curiosity: is it somehow a floating behaviour in the affected versions? because we already had multiple main packages but the bug emerged only after adding a new one

jayconrod · 2019-08-02T14:13:59Z

@andreasph This was a subtle bug that had to be fixed and reopened a couple times as more cases were discovered. I believe there were cases where multiple main packages didn't trigger the bug. In any case, all the fixes were backported to 1.12, so it should be resolved there.

caarlos0 · 2019-10-02T17:57:24Z

still happens on 1.13.1 when running:

go test ./... -v -failfast -race -coverpkg=./... -covermode=atomic -coverprofile=coverage.txt

Changing to go test ./pkg/... etc prevents the issue.

Removing the coverpkg flag also prevents the issue.

Maybe this should be reopen?

bcmills · 2019-10-02T18:05:30Z

@caarlos0, the fix in 1.13 turned out to be incomplete (didn't cover GOPATH mode, which had unexpectedly regressed); see #34223 (comment).

The fix for GOPATH mode is slated to go out in 1.13.2. In the meantime, please try golang.org/dl/gotip or a build from source on release-branch.go1.13 and let us know if it still isn't fixed for your usage.

caarlos0 · 2019-10-02T18:16:59Z

@bcmills worked fine on 0000f0b!

Thanks!

bcmills added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label May 20, 2019

bcmills added this to the Go1.13 milestone May 20, 2019

imsodin changed the title ~~cmd/go: Duplicate symbols with more than one main package in -coverpkg with -coverprofile~~ cmd/go: Duplicate symbols with more than one main package in -coverpkg May 21, 2019

imsodin mentioned this issue May 25, 2019

build: Include cross-package test coverage syncthing/syncthing#5735

Merged

jayconrod mentioned this issue May 28, 2019

cmd/go: tests failing on linux-amd64-longtest [1.12 backport] #32295

Closed

jayconrod closed this as completed May 28, 2019

rodrigo-toledo mentioned this issue Sep 5, 2019

cmd/go: Duplicate symbols with more than one main package in -coverpkg #34114

Closed

golang locked and limited conversation to collaborators Oct 1, 2020

gopherbot added the FrozenDueToAge label Oct 1, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

cmd/go: Duplicate symbols with more than one main package in -coverpkg #32150

cmd/go: Duplicate symbols with more than one main package in -coverpkg #32150

imsodin commented May 20, 2019 •

edited

bcmills commented May 20, 2019

imsodin commented May 21, 2019

jayconrod commented May 21, 2019

jayconrod commented May 28, 2019

gopherbot commented Jun 7, 2019

andreasph commented Aug 1, 2019 •

edited

bcmills commented Aug 1, 2019

andreasph commented Aug 2, 2019 •

edited

jayconrod commented Aug 2, 2019

caarlos0 commented Oct 2, 2019

bcmills commented Oct 2, 2019

caarlos0 commented Oct 2, 2019 •

edited

cmd/go: Duplicate symbols with more than one main package in -coverpkg #32150

cmd/go: Duplicate symbols with more than one main package in -coverpkg #32150

Comments

imsodin commented May 20, 2019 • edited

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

Does this issue reproduce with the latest release?

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

What did you do?

What did you expect to see?

What did you see instead?

bcmills commented May 20, 2019

imsodin commented May 21, 2019

jayconrod commented May 21, 2019

jayconrod commented May 28, 2019

gopherbot commented Jun 7, 2019

andreasph commented Aug 1, 2019 • edited

bcmills commented Aug 1, 2019

andreasph commented Aug 2, 2019 • edited

jayconrod commented Aug 2, 2019

caarlos0 commented Oct 2, 2019

bcmills commented Oct 2, 2019

caarlos0 commented Oct 2, 2019 • edited

imsodin commented May 20, 2019 •

edited

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

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

andreasph commented Aug 1, 2019 •

edited

andreasph commented Aug 2, 2019 •

edited

caarlos0 commented Oct 2, 2019 •

edited