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

Issue 14283043: code review 14283043: go.tools/vcs: allow compilation with go 1.0

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by mreiferson
Modified:
12 years, 1 month ago
Reviewers:
CC:
golang-dev, adg
Visibility:
Public.

Description

go.tools/vcs: allow compilation with go 1.0 It would be nice to be able to use this package as a dependency (or other go utilities in the ecosystem that depend on this package) in environments which have not (or cannot) for whatever reason upgraded to newer versions of golang.

Patch Set 1 #

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

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

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

Messages

Total messages: 5
mreiferson
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go.tools
12 years, 1 month ago (2013-10-02 15:10:27 UTC) #1
adg
Have you signed the Contributor License Agreement? http://golang.org/doc/contribute.html#copyright
12 years, 1 month ago (2013-10-03 02:59:53 UTC) #2
adg
Normally we wouldn't bother keeping go.tools compatible with old versions of Go, but since this ...
12 years, 1 month ago (2013-10-03 03:01:07 UTC) #3
mreiferson
Thanks, Just signed the contributor agreement. I understand, it just seems like it's worthwhile if ...
12 years, 1 month ago (2013-10-03 12:05:16 UTC) #4
adg
12 years, 1 month ago (2013-10-04 01:47:07 UTC) #5
*** Submitted as https://code.google.com/p/go/source/detail?r=43a281dd07af&repo=tools *** go.tools/vcs: allow compilation with go 1.0 It would be nice to be able to use this package as a dependency (or other go utilities in the ecosystem that depend on this package) in environments which have not (or cannot) for whatever reason upgraded to newer versions of golang. R=golang-dev, adg CC=golang-dev https://codereview.appspot.com/14283043 Committer: Andrew Gerrand <adg@golang.org>
Sign in to reply to this message.

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