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

Issue 5595044: code review 5595044: cmd/go: add go tools to rearrangement (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 2 months ago by r
Modified:
13 years, 2 months ago
Reviewers:
CC:
golang-dev, rsc
Visibility:
Public.

Description

cmd/go: add go tools to rearrangement fix, vet yacc is also fixed (it was wrong before) All that's left is the commands used during compilation This looks like a huge CL, but it's almost all file renames. The action is in cmd/go/pkg.go, the Makefiles, and .../doc.go.

Patch Set 1 #

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

Total comments: 1

Patch Set 3 : diff -r aad272ba37c8 https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-47 lines, --60 lines) Patch
M src/cmd/Makefile View 1 chunk +2 lines, -1 line 0 comments Download
M src/cmd/fix/Makefile View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/fix/doc.go View 1 chunk +14 lines, -14 lines 0 comments Download
M src/cmd/fix/error.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/error_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/filepath.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/filepath_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/fix.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/go1pkgrename.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/go1pkgrename_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/googlecode.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/googlecode_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/hashsum.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/hashsum_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/hmacnew.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/hmacnew_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/htmlerr.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/htmlerr_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpfinalurl.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpfinalurl_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpfs.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpfs_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpheaders.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpheaders_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpserver.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httpserver_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httputil.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/httputil_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagecolor.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagecolor_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagenew.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagenew_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagetiled.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imagetiled_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imageycbcr.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/imageycbcr_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/import_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/iocopyn.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/iocopyn_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/main.go View 2 chunks +4 lines, -4 lines 0 comments Download
M src/cmd/fix/main_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/mapdelete.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/mapdelete_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/math.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/math_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/netdial.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/netdial_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/netudpgroup.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/netudpgroup_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/oserrorstring.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/oserrorstring_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/osopen.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/osopen_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/procattr.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/procattr_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/reflect.go View 1 chunk +2 lines, -2 lines 0 comments Download
M src/cmd/fix/reflect_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/signal.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/signal_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/sorthelpers.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/sorthelpers_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/sortslice.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/sortslice_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/strconv.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/strconv_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/stringssplit.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/stringssplit_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/template.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/template_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.asn1.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.asn1.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.datafmt.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.datafmt.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.decode.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.decode.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.decoder.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.decoder.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.dnsmsg.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.dnsmsg.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.encode.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.encode.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.encoder.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.encoder.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.export.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.export.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.print.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.print.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.quick.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.quick.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.read.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.read.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.scan.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.scan.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.script.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.script.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.template.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.template.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.type.go.in View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/testdata/reflect.type.go.out View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/timefileinfo.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/timefileinfo_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/typecheck.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/url.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/url_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/xmlapi.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/fix/xmlapi_test.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/cmd/go/fix.go View 1 2 1 chunk +3 lines, -3 lines 0 comments Download
M src/cmd/go/pkg.go View 1 2 2 chunks +13 lines, -1 line 0 comments Download
M src/cmd/go/vet.go View 1 2 1 chunk +3 lines, -3 lines 0 comments Download
M src/cmd/vet/Makefile View 1 chunk +3 lines, -3 lines 0 comments Download
M src/cmd/vet/doc.go View 2 chunks +4 lines, -4 lines 0 comments Download
M src/cmd/vet/main.go View 2 chunks +2 lines, -2 lines 0 comments Download
M src/cmd/vet/method.go View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/vet/print.go View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/vet/structtag.go View 0 chunks +-1 lines, --1 lines 0 comments Download
M src/pkg/Makefile View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3
r
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go/
13 years, 2 months ago (2012-01-29 18:32:56 UTC) #1
rsc
LGTM http://codereview.appspot.com/5595044/diff/2001/src/cmd/go/pkg.go File src/cmd/go/pkg.go (right): http://codereview.appspot.com/5595044/diff/2001/src/cmd/go/pkg.go#newcode271 src/cmd/go/pkg.go:271: if t.Goroot && isGoTool[elem] { I am a ...
13 years, 2 months ago (2012-01-29 18:51:10 UTC) #2
r
13 years, 2 months ago (2012-01-29 19:07:29 UTC) #3
*** Submitted as 80b6f579af56 ***

cmd/go: add go tools to rearrangement
fix, vet
yacc is also fixed (it was wrong before)
All that's left is the commands used during compilation
This looks like a huge CL, but it's almost all file renames.
The action is in cmd/go/pkg.go, the Makefiles, and .../doc.go.

R=golang-dev, rsc
CC=golang-dev
http://codereview.appspot.com/5595044
Sign in to reply to this message.

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