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

x/debug/cmd/viewcore: does not build on aix-ppc64 #32839

Open
bcmills opened this issue Jun 28, 2019 · 8 comments
Open

x/debug/cmd/viewcore: does not build on aix-ppc64 #32839

bcmills opened this issue Jun 28, 2019 · 8 comments
Labels
Builders x/build issues (builders, bots, dashboards) help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-AIX Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone

Comments

@bcmills
Copy link
Contributor

bcmills commented Jun 28, 2019

The x/debug build is consistently failing on the aix-ppc64 builder.
It appears that x/debug depends on github.com/chzyer/readline, but that package is missing an aix implementation.

CC @chzyer @Helflym

aix-ppc64 at 67f181bfd84dfd5942fe9a29d8a20c9ce5eb2fea building debug at 621e2d3f35dac46daf912f8d9d2fabd57d022520

:: Running /ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/bin/go with args ["/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/bin/go" "test" "-short" "golang.org/x/debug/..."] and env ["TERM=dumb" "AUTHSTATE=compat" "SHELL=/usr/bin/ksh" "HOME=/" "USER=root" "PATH=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/bin:/opt/freeware/bin:/usr/bin:/etc:/usr/sbin:/usr/ucb:/usr/bin/X11:/sbin:/usr/java7_64/jre/bin:/usr/java7_64/bin" "TZ=CST6CDT" "LANG=en_US" "LOCPATH=/usr/lib/nls/loc" "LC__FASTMSG=true" "ODMDIR=/etc/objrepos" "CLCMD_PASSTHRU=1" "LOGNAME=root" "LOGIN=root" "META_BUILDLET_BINARY_URL=https://storage.googleapis.com/go-builder-data/buildlet.aix-ppc64" "GO_STAGE0_NET_DELAY=300ms" "GO_STAGE0_DL_DELAY=100ms" "NLSPATH=/usr/lib/nls/msg/%L/%N:/usr/lib/nls/msg/%L/%N.cat:/usr/lib/nls/msg/%l.%c/%N:/usr/lib/nls/msg/%l.%c/%N.cat" "WORKDIR=/ramdisk8GB/workdir-host-aix-ppc64-osuosl" "GOROOT_BOOTSTRAP=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go1.4" "GO_BUILDER_NAME=aix-ppc64" "GOROOT_BOOTSTRAP=/opt/freeware/lib/golang" "PATH=/opt/freeware/bin:/usr/bin:/etc:/usr/sbin:/usr/ucb:/usr/bin/X11:/sbin:/usr/java7_64/jre/bin:/usr/java7_64/bin" "GOROOT=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go" "GOPATH=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/gopath" "GOPROXY=http://10.240.0.23:30157" "GOPROXY=https://proxy.golang.org" "TMPDIR=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/tmp" "GOCACHE=/ramdisk8GB/workdir-host-aix-ppc64-osuosl/gocache"] in dir /ramdisk8GB/workdir-host-aix-ppc64-osuosl/gopath/src/golang.org/x/debug

go: downloading github.com/spf13/cobra v0.0.3
go: downloading github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e
go: extracting github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e
go: extracting github.com/spf13/cobra v0.0.3
go: downloading github.com/spf13/pflag v1.0.3
go: extracting github.com/spf13/pflag v1.0.3
go: finding github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e
go: finding github.com/spf13/pflag v1.0.3
go: finding github.com/spf13/cobra v0.0.3
# github.com/chzyer/readline
../../../../pkg/mod/github.com/chzyer/readline@v0.0.0-20180603132655-2972be24d48e/utils.go:241:9: undefined: State
ok  	golang.org/x/debug/internal/core	0.031s
@bcmills bcmills added help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-AIX labels Jun 28, 2019
@gopherbot gopherbot added this to the Unreleased milestone Jun 28, 2019
@bcmills bcmills added the Builders x/build issues (builders, bots, dashboards) label Jun 28, 2019
@bcmills
Copy link
Contributor Author

bcmills commented Jun 28, 2019

@golang/osp-team, should we simply disable the aix-ppc64 on this repo for now?

@dmitshur dmitshur added the Testing An issue that has been verified to require only test changes, not just a test failure. label Jun 28, 2019
@dmitshur
Copy link
Contributor

In #31564 (comment), @Helflym said "my aim is to have all x/ packages working for 1.13." If that still holds and applies to x/debug, we can do the same as for x/net and make the builders require Go 1.13. That'll buy time, and if we want to disable it completely later, we can do that.

@dmitshur
Copy link
Contributor

Doing what I said above is trickier than I thought. The current version of cmd/coordinator has atLeastGo1 helper to conditionally limit to a specific version of Go. But it doesn't exclude master branch. I understand atLeastGo1 was designed to cut off too-old versions of Go, not to target not-yet-released future versions of Go.

The difference with x/net is that x/net is passing on master branch, but not Go 1.12 release branch. But x/debug is failing on all branches, including master.

So for now our option is to disable it completely and update that as x/debug gains aix-ppc64 support. I'll wait a bit to give aix-ppc64 owners a chance to comment on their plans for x/debug support before sending that CL.

@Helflym
Copy link
Contributor

Helflym commented Jul 1, 2019

I've already created a PR in chzyer/readline package in order to add AIX support (chzyer/readline#175). However, this package seems no longer supported..
This is the only problem with this package (at least locally).

Note that if it's happening with AIX, it might happen to every new GOOS/GOARCH. I'm not sure having a subrepos package depending on a external package is a good idea. If we want all subrepos to be green, we should be able to submit patches and fixes directly inside Gerrit.

Anyway, if you wish to disable it until either the PR is accepted or a better solution is created, go for it.

@bcmills
Copy link
Contributor Author

bcmills commented Jul 1, 2019

The dependency comes from cmd/viewcore/main.go in CL 115597 (CC @hyangah, @randall77, @heschik).

@gopherbot
Copy link

Change https://golang.org/cl/192322 mentions this issue: cmd/viewcore: skip build on aix

@bcmills bcmills changed the title x/debug: consistent build failure on aix-ppc64 x/debug/cmd/viewcore: does not build on aix-ppc64 Aug 29, 2019
gopherbot pushed a commit to golang/debug that referenced this issue Aug 29, 2019
./cmd/viewcore depends on github.com/chzyer/readline,
which doesn't build on aix yet.

Updates golang/go#32839

Change-Id: I918667fcbc48a2ed5e8bb550be962932ccc8131d
Reviewed-on: https://go-review.googlesource.com/c/debug/+/192322
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
@millerresearch
Copy link
Contributor

github.com/chzyer/readline doesn't build on Plan 9 either. I'll submit a CL with a +build !plan9 to stop the x/debug repo builder tests failing on Plan 9.

@gopherbot
Copy link

Change https://golang.org/cl/229877 mentions this issue: cmd/viewcore: skip build on plan9

gopherbot pushed a commit to golang/debug that referenced this issue Apr 24, 2020
./cmd/viewcore depends on github.com/chzyer/readline, which
doesn't build on Plan 9.

Updates golang/go#32839

Change-Id: I5e0638a65914dcad7e7c9383df77e24a6b616a89
Reviewed-on: https://go-review.googlesource.com/c/debug/+/229877
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Builders x/build issues (builders, bots, dashboards) help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-AIX Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Development

No branches or pull requests

5 participants