Skip to content

Commit

Permalink
cmd/release: remove code that removes nonexistent oldlink binary
Browse files Browse the repository at this point in the history
The oldlink binary is not present in the Go 1.16 source tree,
the oldest Go major version we still make minor releases for.
Undo the change added in CL 242643 as it is no longer needed.

Updates golang/go#39509.

Change-Id: I83ef35de022641eb354487600b3b6d705fd9bcf8
Reviewed-on: https://go-review.googlesource.com/c/build/+/384715
Trust: Dmitri Shuralyov <[email protected]>
Run-TryBot: Dmitri Shuralyov <[email protected]>
Auto-Submit: Dmitri Shuralyov <[email protected]>
TryBot-Result: Gopher Robot <[email protected]>
Reviewed-by: Cherry Mui <[email protected]>
  • Loading branch information
dmitshur authored and gopherbot committed Feb 11, 2022
1 parent 701a7ea commit 8a3cec9
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions cmd/release/release.go
Original file line number Diff line number Diff line change
Expand Up @@ -523,14 +523,6 @@ func (b *Build) make() error {
if err := client.RemoveAll(ctx, b.toolDir()+"/api"); err != nil {
return err
}
// The oldlink tool is used for debugging differences during
// testing of the linker rewrite, and can be built by users
// if necessary. See issue 39509.
// This can be removed when oldlink is gone, likely once Go 1.16
// is no longer supported.
if err := client.RemoveAll(ctx, b.toolDir()+"/oldlink"); err != nil {
return err
}
// Remove go/pkg/${GOOS}_${GOARCH}/cmd. This saves a bunch of
// space, and users don't typically rebuild cmd/compile,
// cmd/link, etc. If they want to, they still can, but they'll
Expand Down

0 comments on commit 8a3cec9

Please sign in to comment.