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
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
I think those who only have clang (e.g. FreeBSD) can modify the file fairly
easily
(just change the GCC=gcc line), and this script is only intend to be run by
a small group of people, so perhaps we don't need to make the change.
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
On Fri, Feb 21, 2014 at 10:14 PM, Mikio Hara <mikioh.mikioh@gmail.com>wrote:
> please say "not lgtm" if you dislike or you think it's not worth.
>
Basically I'm on the fence. Let's wait for others to make the decision.
*** 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
Issue 67170043: code review 67170043: syscall: make mkerrors.sh work with clang
(Closed)
Created 11 years, 4 months ago by mikio
Modified 11 years, 4 months ago
Reviewers: gobot, dave_cheney.net
Base URL:
Comments: 0