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

Issue 1231041: code review 1231041: build; allow MAKEFLAGS to be set outside the build scripts (Closed)

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

Description

build; allow MAKEFLAGS to be set outside the build scripts

Patch Set 1 #

Patch Set 2 : code review 1231041: build; allow MAKEFLAGS to be set outside the build scripts #

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

Messages

Total messages: 3
cw
Hello rsc, r (cc: golang-dev@googlegroups.com), I'd like you to review this change.
14 years, 10 months ago (2010-05-14 23:00:14 UTC) #1
r
LGTM
14 years, 10 months ago (2010-05-15 17:07:58 UTC) #2
r
14 years, 10 months ago (2010-05-15 17:08:37 UTC) #3
*** Submitted as http://code.google.com/p/go/source/detail?r=470cdecaba63 ***

build; allow MAKEFLAGS to be set outside the build scripts

R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1231041

Committer: Rob Pike <r@golang.org>
Sign in to reply to this message.

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