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

Issue 608041: code review 608041: runtime: add GOROOT and Version (Closed)

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

Description

runtime: add GOROOT and Version

Patch Set 1 #

Patch Set 2 : code review 608041: runtime: add GOROOT and Version #

Patch Set 3 : code review 608041: runtime: add GOROOT and Version #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -2 lines) Patch
M .hgignore View 1 chunk +5 lines, -1 line 0 comments Download
M src/pkg/runtime/Makefile View 3 chunks +8 lines, -1 line 0 comments Download
M src/pkg/runtime/extern.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
A src/pkg/runtime/mkversion.c View 1 chunk +15 lines, -0 lines 0 comments Download
M src/pkg/runtime/runtime.c View 1 2 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 6
rsc
Hello r (cc: golang-dev@googlegroups.com), I'd like you to review this change.
15 years, 4 months ago (2010-03-17 00:28:22 UTC) #1
r
where is getgoversion?
15 years, 4 months ago (2010-03-17 04:50:36 UTC) #2
rsc
On Tue, Mar 16, 2010 at 21:50, <r@golang.org> wrote: > where is getgoversion? it's in ...
15 years, 4 months ago (2010-03-17 05:31:21 UTC) #3
rsc
... and getgoversion returns the string printed by src/version.bash, built into lib9 during the make ...
15 years, 4 months ago (2010-03-17 05:31:56 UTC) #4
r
LGTM
15 years, 4 months ago (2010-03-17 05:34:23 UTC) #5
rsc
15 years, 4 months ago (2010-03-17 06:10:36 UTC) #6
*** Submitted as http://code.google.com/p/go/source/detail?r=641ed035545d ***

runtime: add GOROOT and Version

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

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