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

Issue 67170043: code review 67170043: syscall: make mkerrors.sh work with clang (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 4 months ago by mikio
Modified:
11 years, 4 months ago
Reviewers:
gobot, dave, iant
CC:
golang-codereviews, minux1, iant
Visibility:
Public.

Description

syscall: make mkerrors.sh work with clang

Patch Set 1 : diff -r 70499e5fbe5b https://code.google.com/p/go #

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -7 lines) Patch
M src/pkg/syscall/mkerrors.sh View 4 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 9
mikio
Hello golang-codereviews@googlegroups.com (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
11 years, 4 months ago (2014-02-22 01:23:07 UTC) #1
minux1
I think those who only have clang (e.g. FreeBSD) can modify the file fairly easily ...
11 years, 4 months ago (2014-02-22 02:55:07 UTC) #2
mikio
please say "not lgtm" if you dislike or you think it's not worth.
11 years, 4 months ago (2014-02-22 03:14:54 UTC) #3
minux1
On Fri, Feb 21, 2014 at 10:14 PM, Mikio Hara <mikioh.mikioh@gmail.com>wrote: > please say "not ...
11 years, 4 months ago (2014-02-22 03:23:38 UTC) #4
gobot
R=iant@golang.org (assigned by minux.ma@gmail.com)
11 years, 4 months ago (2014-02-22 03:24:38 UTC) #5
iant
LGTM This is a minor change but I think it's fine.
11 years, 4 months ago (2014-02-24 05:55:29 UTC) #6
mikio
*** Submitted as https://code.google.com/p/go/source/detail?r=3cf533be5e36 *** syscall: make mkerrors.sh work with clang LGTM=iant R=golang-codereviews, minux.ma, gobot, ...
11 years, 4 months ago (2014-02-24 07:34:57 UTC) #7
gobot
This CL appears to have broken the darwin-386-cheney builder.
11 years, 4 months ago (2014-02-24 07:39:13 UTC) #8
dave_cheney.net
11 years, 4 months ago (2014-02-24 07:53:17 UTC) #9
This is a real but unrelated failure, Josh has opened a bug. 

> On 24 Feb 2014, at 18:39, gobot@golang.org wrote:
> 
> This CL appears to have broken the darwin-386-cheney builder.
> 
> https://codereview.appspot.com/67170043/
> 
> -- 
> You received this message because you are subscribed to the Google Groups
"golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
Sign in to reply to this message.

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