Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: new internal/goroot test broke aix, illumos, solaris builders #34390

Closed
ALTree opened this issue Sep 19, 2019 · 3 comments
Closed

cmd/go: new internal/goroot test broke aix, illumos, solaris builders #34390

ALTree opened this issue Sep 19, 2019 · 3 comments
Milestone

Comments

@ALTree
Copy link
Member

@ALTree ALTree commented Sep 19, 2019

The new internal/goroot mod_m test introduced in CL 196120 with the fix for #34358 is currently failing on the aix, illumos, and solaris builders, with the following error:

--- FAIL: TestScript (0.01s)
    --- FAIL: TestScript/mod_m (4.13s)
        script_test.go:191: 
            # It's absurd, but builds with -compiler=gccgo used to fail to build module m.
            # golang.org/issue/34358 (3.875s)
            > cd m
            $WORK/gopath/src/m
            > go build
            > exists m$GOEXE
            > rm m$GOEXE
            > [exec:gccgo] go build -compiler=gccgo
            [stderr]
            # m
            ld: 0711-317 ERROR: Undefined symbol: .runtime.setmodinfo
            ld: 0711-345 Use the -bloadmap or -bnoquiet option to obtain more information.
            collect2: error: ld returned 8 exit status
            [exit status 2]
            FAIL: testdata/script/mod_m.txt:8: unexpected command failure
            
FAIL
FAIL	cmd/go	107.473s
FAIL

cc @ianlancetaylor

@ALTree ALTree added this to the Go1.14 milestone Sep 19, 2019
@Helflym

This comment has been minimized.

Copy link
Contributor

@Helflym Helflym commented Sep 19, 2019

I think this is linked with CL 196237. Symbol runtime.setmodinfo is only available in gcc-master with go1.13. In previous gcc version thus previous Go version, runtime/debug.setmodinfo is still linked to runtime..z2fdebug.setmodinfo.

gcc-9.1.0:

go/runtime/debug/mod.go://go:linkname setmodinfo runtime..z2fdebug.setmodinfo

gcc-svn:

go/runtime/debug.go://go:linkname setmodinfo runtime.setmodinfo

Thus, gccgo version must be checked when creating _gomod_.go.

AIX builder is still in gcc 8 (for various reasons) and I think it's the same for Illumos and Solaris builders.

@av86743

This comment has been minimized.

Copy link

@av86743 av86743 commented Sep 19, 2019

Are trybots' gccgo/gcc out of sync with builders?

@ALTree

This comment has been minimized.

Copy link
Member Author

@ALTree ALTree commented Sep 19, 2019

@ALTree ALTree closed this Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.