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

Issue 6499053: code review 6499053: codereview: advise users to install 'easy_install'. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 8 months ago by adonovan
Modified:
11 years, 3 months ago
Reviewers:
CC:
golang-dev
Visibility:
Public.

Description

codereview: advise users to install 'easy_install'. My first go commit. :)

Patch Set 1 #

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M lib/codereview/codereview.py View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 14
adonovan
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go/
11 years, 8 months ago (2012-08-29 21:21:24 UTC) #1
r
Really? I've found easy-install to not be easy and not install things. -rob
11 years, 8 months ago (2012-08-29 21:44:10 UTC) #2
adonovan
That may be, but if we are advising people to use it, we may as ...
11 years, 8 months ago (2012-08-29 21:52:54 UTC) #3
r
On Wed, Aug 29, 2012 at 2:52 PM, Alan Donovan <adonovan@google.com> wrote: > That may ...
11 years, 8 months ago (2012-08-29 21:53:19 UTC) #4
rh
The Go documentation at http://golang.org/doc/install/source#mercurial suggests downloading Mercurial from http://mercurial.selenic.com/downloads/ Would that make more sense? ...
11 years, 8 months ago (2012-08-30 02:37:11 UTC) #5
rh
(as the only option suggested) On Wednesday, August 29, 2012 9:37:11 PM UTC-5, rh wrote: ...
11 years, 8 months ago (2012-08-30 02:44:35 UTC) #6
r
Yes. Don't use easy_install. Let's fix it that way instead, to agree with the mercurial ...
11 years, 8 months ago (2012-08-30 04:54:45 UTC) #7
rh
adonovan, are you still interested in completing this CL? I'd be happy to, if not.
11 years, 6 months ago (2012-09-29 22:14:43 UTC) #8
rsc
not lgtm We decided not to advise the use of easy_install. It used to be ...
11 years, 6 months ago (2012-10-01 19:16:59 UTC) #9
remyoudompheng
It seems like gocodereview doesn't recognize the lowercase "not 1gtm"
11 years, 6 months ago (2012-10-01 19:36:00 UTC) #10
adonovan
Robert, thanks for your offer; if you'd like to take it, please be my guest. ...
11 years, 6 months ago (2012-10-01 19:38:15 UTC) #11
rh
Thank you. I have created http://codereview.appspot.com/6584059 On 2012/10/01 19:38:15, adonovan wrote: > Robert, thanks for ...
11 years, 6 months ago (2012-10-02 15:35:33 UTC) #12
r
11 years, 6 months ago (2012-10-02 21:51:49 UTC) #13
adonovan
11 years, 3 months ago (2013-01-15 22:10:35 UTC) #14
*** Abandoned ***
Sign in to reply to this message.

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