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

Issue 4654045: code review 4654045: goinstall: undo 1ad616fb313d (always rebuild...) (Closed)

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

Description

goinstall: undo 1ad616fb313d (always rebuild...) CL 4627051 is a better way of doing the same thing.

Patch Set 1 #

Patch Set 2 : diff -r 812bb69a9ec1 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -7 lines) Patch
M src/cmd/goinstall/main.go View 3 chunks +3 lines, -7 lines 0 comments Download

Messages

Total messages: 3
adg
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg/
13 years, 9 months ago (2011-06-21 07:06:35 UTC) #1
r
LGTM but CL desc should say why
13 years, 9 months ago (2011-06-21 07:12:38 UTC) #2
adg
13 years, 9 months ago (2011-06-21 07:13:26 UTC) #3
*** Submitted as http://code.google.com/p/go/source/detail?r=1cad1e8470ba ***

goinstall: undo 1ad616fb313d (always rebuild...)

CL 4627051 is a better way of doing the same thing.

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

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