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

all: remove darwin/386 #37610

Closed
andybons opened this issue Mar 2, 2020 · 12 comments
Closed

all: remove darwin/386 #37610

andybons opened this issue Mar 2, 2020 · 12 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. release-blocker
Milestone

Comments

@andybons
Copy link
Member

andybons commented Mar 2, 2020

Per #34749, we’re going to remove darwin/386 for Go 1.15. This is a tracking issue for that work.

Some affected files (taken from above issue. Thanks, @bradfitz):

./runtime/rt0_darwin_386.s
./runtime/cgo/gcc_darwin_386.c
./runtime/signal_darwin_386.go
./runtime/defs_darwin_386.go
./runtime/sys_darwin_386.s
./debug/macho/testdata/gcc-386-darwin-exec
./debug/macho/testdata/fat-gcc-386-amd64-darwin-exec
./debug/macho/testdata/clang-386-darwin.obj
./debug/macho/testdata/clang-386-darwin-exec-with-rpath
./cmd/vendor/golang.org/x/sys/unix/zsyscall_darwin_386.1_11.go
./cmd/vendor/golang.org/x/sys/unix/asm_darwin_386.s
./cmd/vendor/golang.org/x/sys/unix/zerrors_darwin_386.go
./cmd/vendor/golang.org/x/sys/unix/syscall_darwin_386.go
./cmd/vendor/golang.org/x/sys/unix/zsyscall_darwin_386.s
./cmd/vendor/golang.org/x/sys/unix/zsysnum_darwin_386.go
./cmd/vendor/golang.org/x/sys/unix/zsyscall_darwin_386.go
./cmd/vendor/golang.org/x/sys/unix/ztypes_darwin_386.go
./syscall/asm_darwin_386.s
./syscall/zerrors_darwin_386.go
./syscall/syscall_darwin_386.go
./syscall/zsyscall_darwin_386.s
./syscall/zsysnum_darwin_386.go
./syscall/zsyscall_darwin_386.go
./syscall/ztypes_darwin_386.go

But also stuff in debug/macho (see below comment), cmd/link, etc.

@andybons andybons added NeedsFix The path to resolution is known, but the work has not been done. release-blocker labels Mar 2, 2020
@andybons andybons added this to the Go1.15 milestone Mar 2, 2020
@andybons andybons self-assigned this Mar 2, 2020
@ianlancetaylor
Copy link
Member

Just a note that I don't think we should remove any 386 support from debug/macho. The debug/macho package is for analyzing Mach-O files, and we should continue to support examining Mach-O 386 files even if we can no longer generate them.

@andybons
Copy link
Member Author

andybons commented Mar 2, 2020

@ianlancetaylor sgtm. Thanks for the note.

@andybons andybons assigned aclements and unassigned andybons Mar 9, 2020
@FiloSottile
Copy link
Contributor

FiloSottile commented Apr 3, 2020

Can we drop the darwin/386 builders from tip? I am failing them in CL 227037 but it's not worth fixing them since we are dropping the target.

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/226985 mentions this issue: cmd/dist: remove darwin/386 as a valid port

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227181 mentions this issue: dashboard: update misc-compile--darwin description

@dmitshur
Copy link
Contributor

dmitshur commented Apr 4, 2020

@FiloSottile Yes, we should stop running those builders on tip (but they should still run on 1.14 and older release branches).

The misc-compile-darwin builder get its list of exact GOOS/GOARCH pairs from go tool dist list output (see here and here). So it'll stop being tested on as soon as darwin/386 is removed from tip.

I see @bradfitz already sent CL 226985 for that. If you want, you could rebase your CL on top to test it.

gopherbot pushed a commit that referenced this issue Apr 4, 2020
This only removes the ability to build it, and removes it as a
src/buildall.bash target (which uses go tool dist list).

Now:

$ go tool dist list | grep ^darwin
darwin/amd64
darwin/arm64

After this, remaining is removing leftover port--specific code in the
tree.

Updates #37610
Updates #37611

Change-Id: I00f03b2355c2e152f75e57abd3063be243529d2d
Reviewed-on: https://go-review.googlesource.com/c/go/+/226985
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Filippo Valsorda <filippo@golang.org>
gopherbot pushed a commit to golang/build that referenced this issue Apr 6, 2020
The 32-bit pairs darwin/386 and darwin/arm are removed in Go 1.15,
and the master branch is where Go 1.15 is being developed.
The builder has started to fail after CL 226985 (as intended),
so remove it. Keep it running on Go 1.14 and 1.13 release branches.

Also update the misc-compile-darwin builder description.

For golang/go#37610.
For golang/go#37611.

Change-Id: Ia70386c5ae2821b4397048dcb0862a066463d6de
Reviewed-on: https://go-review.googlesource.com/c/build/+/227181
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Andrew Bonventre <andybons@golang.org>
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227582 mentions this issue: all: remove scattered remnants of darwin/386

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227581 mentions this issue: all: remove darwin/386 build-tags and files

gopherbot pushed a commit that referenced this issue Apr 8, 2020
This removes all files that are only used on darwin/386 and cleans up
build tags in files that are still used on other platforms.

Updates #37610.

Change-Id: If246642476c12d15f59a474e2b91a29c0c02fe75
Reviewed-on: https://go-review.googlesource.com/c/go/+/227581
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227552 mentions this issue: go/packages: exclude darwin port from TestSizes test

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227763 mentions this issue: [dev.link] cmd/link: delete darwin/386 host object loading support

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/227766 mentions this issue: cmd/oldlink: delete darwin/386 host object loading support

gopherbot pushed a commit that referenced this issue Apr 9, 2020
In Go 1.15, the darwin/386 port is gone.

Updates #37610.

Change-Id: I11627c03403d4d4a3e0950fdedd0d850ae24b439
Reviewed-on: https://go-review.googlesource.com/c/go/+/227766
Run-TryBot: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
gopherbot pushed a commit that referenced this issue Apr 9, 2020
In Go 1.15, the darwin/386 port is gone.

Updates #37610.

Change-Id: I308561848d90d9a649b3603ab1f7fc1117305ec1
Reviewed-on: https://go-review.googlesource.com/c/go/+/227763
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
gopherbot pushed a commit to golang/tools that referenced this issue Apr 9, 2020
The darwin/386 port has been removed per golang.org/issue/37610.
TestSizes needs an operating system that has both amd64 and 386 ports,
so darwin no longer qualifies. NetBSD has had its 386 port restored
recently in golang/go#31726, so it can be used instead.

Fixes golang/go#38319.
For golang/go#37610.

Change-Id: I37ce6d86ca3ddad43e9294e0de66f36091cba54a
Reviewed-on: https://go-review.googlesource.com/c/tools/+/227552
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/229323 mentions this issue: [dev.link] cmd/link, cmd/oldlink: remove remaining darwin/386 and darwin/arm code

gopherbot pushed a commit that referenced this issue Apr 22, 2020
…rm code

Updates #37610, #37611.

Change-Id: I0a497af03e24ddea40ed3e342f3a9362bf21ac0c
Reviewed-on: https://go-review.googlesource.com/c/go/+/229323
Run-TryBot: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Austin Clements <austin@google.com>
xujianhai666 pushed a commit to xujianhai666/go-1 that referenced this issue May 21, 2020
…rm code

Updates golang#37610, golang#37611.

Change-Id: I0a497af03e24ddea40ed3e342f3a9362bf21ac0c
Reviewed-on: https://go-review.googlesource.com/c/go/+/229323
Run-TryBot: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Austin Clements <austin@google.com>
eclipseo added a commit to eclipseo/go-daemon that referenced this issue Aug 23, 2020
sharadg pushed a commit to sharadg/kubernetes that referenced this issue Oct 23, 2020
Removed in go1.15 (GH/golang/go/issues/37610).

Signed-off-by: Stephen Augustus <saugustus@vmware.com>
zquestz added a commit to gcash/bchd that referenced this issue Oct 24, 2020
johnboyes added a commit to agilepathway/gauge-jira that referenced this issue Jan 16, 2021
[Go no longer supports darwin/386 (MacOS 32-bit apps), as of Go 1.15][1]

[1]: golang/go#37610
johnboyes added a commit to agilepathway/gauge-jira that referenced this issue Jan 16, 2021
[Go no longer supports darwin/386 (MacOS 32-bit apps), as of Go 1.15][1]

[1]: golang/go#37610
johnboyes added a commit to agilepathway/gauge-jira that referenced this issue Jan 16, 2021
[Go no longer supports darwin/386 (MacOS 32-bit apps), as of Go 1.15][1]

[1]: golang/go#37610
bengadbois added a commit to bengadbois/pewpew that referenced this issue Jan 20, 2021
bengadbois added a commit to bengadbois/pewpew that referenced this issue Jan 20, 2021
johnboyes added a commit to agilepathway/gauge-jira that referenced this issue Feb 1, 2021
[Go no longer supports darwin/386 (MacOS 32-bit apps), as of Go 1.15][1]

[1]: golang/go#37610
peat-psuwit added a commit to peat-psuwit/aptly that referenced this issue Mar 24, 2021
Go 1.15 drops support for darwin/386 GOOS/GOARCH pair [1]. So, we have
to skip this pair, and thus cannot use simple os multiply arch anymore.
Switch to goxc's BuildConstraints config, which uses the same syntax as
Go's build constraint header [2].

[1] golang/go#37610
[2] https://golang.org/cmd/go/#hdr-Build_constraints
peat-psuwit added a commit to peat-psuwit/aptly that referenced this issue Mar 26, 2021
Go 1.15 drops support for darwin/386 GOOS/GOARCH pair [1]. So, we have
to skip this pair, and thus cannot use simple os multiply arch anymore.
Switch to goxc's BuildConstraints config, which uses the same syntax as
Go's build constraint header [2].

[1] golang/go#37610
[2] https://golang.org/cmd/go/#hdr-Build_constraints
@golang golang locked and limited conversation to collaborators Apr 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. release-blocker
Projects
None yet
Development

No branches or pull requests

6 participants