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

Issue 4431064: code review 4431064: install: convert GOROOT and GOBIN Windows-style paths t...

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

Description

install: convert GOROOT and GOBIN Windows-style paths to Unix-like paths GOROOT and GOBIN paths are used to build the Go toolchain and programs. To build a Go program on Windows, the paths should be valid Windows full paths, starting with the drive letter and a colon e.g. C:\go\bin or c:/go/bin. On Windows, the Go toolchain is built using the MSYS/MinGW Unix-like toolchain. MSYS knows that it must convert a Windows-like PATH to a Unix-like path. MSYS does not know that it should convert Windows-like GOROOT and GOBIN paths to Unix-like paths, so we have to do the conversion. Rob Pike's windows_path function has been revised to convert Windows drive letters to lowercase. Fixes issue 1734. Fixes issue 1735.

Patch Set 1 #

Patch Set 2 : diff -r 63dd23bf8663 https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 63dd23bf8663 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -9 lines) Patch
M src/env.bash View 1 1 chunk +11 lines, -9 lines 0 comments Download

Messages

Total messages: 3
peterGo
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
13 years ago (2011-04-25 15:43:32 UTC) #1
rsc
Thanks for fixing this, but there are too many CLs about this topic. Instead of ...
13 years ago (2011-04-25 15:46:41 UTC) #2
rsc
11 years, 11 months ago (2012-06-03 04:50:47 UTC) #3

          
Sign in to reply to this message.

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