Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(689)

Issue 106740043: code review 106740043: [release-branch.go1.3] remove cmd/link (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by adg
Modified:
9 years, 11 months ago
Reviewers:
rsc, bradfitz
CC:
golang-codereviews
Visibility:
Public.

Description

[release-branch.go1.3] remove cmd/link

Patch Set 1 #

Patch Set 2 : diff -r de014d3583bb https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5268 lines) Patch
R src/cmd/link/auto.go View 1 chunk +0 lines, -117 lines 0 comments Download
R src/cmd/link/auto_test.go View 1 chunk +0 lines, -72 lines 0 comments Download
R src/cmd/link/dead.go View 1 chunk +0 lines, -74 lines 0 comments Download
R src/cmd/link/dead_test.go View 1 chunk +0 lines, -97 lines 0 comments Download
R src/cmd/link/debug.go View 1 chunk +0 lines, -11 lines 0 comments Download
R src/cmd/link/hex_test.go View 1 chunk +0 lines, -74 lines 0 comments Download
R src/cmd/link/layout.go View 1 chunk +0 lines, -180 lines 0 comments Download
R src/cmd/link/layout_test.go View 1 chunk +0 lines, -45 lines 0 comments Download
R src/cmd/link/link_test.go View 1 chunk +0 lines, -35 lines 0 comments Download
R src/cmd/link/load.go View 1 chunk +0 lines, -116 lines 0 comments Download
R src/cmd/link/macho.go View 1 chunk +0 lines, -380 lines 0 comments Download
R src/cmd/link/macho_test.go View 1 chunk +0 lines, -407 lines 0 comments Download
R src/cmd/link/main.go View 1 chunk +0 lines, -9 lines 0 comments Download
R src/cmd/link/pclntab.go View 1 chunk +0 lines, -479 lines 0 comments Download
R src/cmd/link/pclntab_test.go View 1 chunk +0 lines, -334 lines 0 comments Download
R src/cmd/link/prog.go View 1 chunk +0 lines, -220 lines 0 comments Download
R src/cmd/link/prog_test.go View 1 chunk +0 lines, -163 lines 0 comments Download
R src/cmd/link/runtime.go View 1 chunk +0 lines, -28 lines 0 comments Download
R src/cmd/link/scan.go View 1 chunk +0 lines, -187 lines 0 comments Download
R src/cmd/link/testdata/Makefile View 1 chunk +0 lines, -15 lines 0 comments Download
R src/cmd/link/testdata/autosection.6 View Binary file 0 comments Download
R src/cmd/link/testdata/autosection.s View 1 chunk +0 lines, -60 lines 0 comments Download
R src/cmd/link/testdata/autoweak.6 View Binary file 0 comments Download
R src/cmd/link/testdata/autoweak.s View 1 chunk +0 lines, -30 lines 0 comments Download
R src/cmd/link/testdata/dead.6 View Binary file 0 comments Download
R src/cmd/link/testdata/dead.s View 1 chunk +0 lines, -49 lines 0 comments Download
R src/cmd/link/testdata/genpcln.go View 1 chunk +0 lines, -112 lines 0 comments Download
R src/cmd/link/testdata/hello.6 View Binary file 0 comments Download
R src/cmd/link/testdata/hello.s View 1 chunk +0 lines, -15 lines 0 comments Download
R src/cmd/link/testdata/layout.6 View Binary file 0 comments Download
R src/cmd/link/testdata/layout.s View 1 chunk +0 lines, -29 lines 0 comments Download
R src/cmd/link/testdata/link.hello.darwin.amd64 View 1 chunk +0 lines, -57 lines 0 comments Download
R src/cmd/link/testdata/macho.amd64.exit9 View 1 chunk +0 lines, -24 lines 0 comments Download
R src/cmd/link/testdata/macho.amd64.hello View 1 chunk +0 lines, -39 lines 0 comments Download
R src/cmd/link/testdata/macho.amd64.helloro View 1 chunk +0 lines, -34 lines 0 comments Download
R src/cmd/link/testdata/pclntab.6 View Binary file 0 comments Download
R src/cmd/link/testdata/pclntab.s View 1 chunk +0 lines, -1751 lines 0 comments Download
R src/cmd/link/util.go View 1 chunk +0 lines, -11 lines 0 comments Download
R src/cmd/link/write.go View 1 chunk +0 lines, -14 lines 0 comments Download

Messages

Total messages: 3
adg
Hello rsc (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
9 years, 11 months ago (2014-06-02 02:24:58 UTC) #1
bradfitz
LGTM On Mon, Jun 2, 2014 at 11:24 AM, <adg@golang.org> wrote: > Reviewers: rsc, > ...
9 years, 11 months ago (2014-06-02 02:31:48 UTC) #2
adg
9 years, 11 months ago (2014-06-02 03:34:50 UTC) #3
Message was sent while issue was closed.
Submitted as
https://code.google.com/p/go/source/detail?r=aee0f76b956217ac9ab5aa75654a080d...

But it looks like this breaks nacl:

mkzip: ../misc/nacl/testzip.proto:16: stat ../src/cmd/link: no such file or
directory
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b