code review 2443041: build: fix darwin/386 build

build: fix darwin/386 build

[adg, 2010-10-12 00:48:22]

Hello rsc (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[adg, 2010-10-12 00:49:09]

*** Submitted as http://code.google.com/p/go/source/detail?r=b725fd0ec942 ***

build: fix darwin/386 build

R=rsc
CC=golang-dev
http://codereview.appspot.com/2443041

[rsc, 2010-10-12 01:52:00]

LGTM

I am confused about why this is the right fix,
but it's clearly equivalent to what was here before,
and if it works, I'm happy to postpone understanding
until the next time Apple changes something.

[adg, 2010-10-12 02:04:39]

On 12 October 2010 12:51, Russ Cox <rsc@golang.org> wrote:
> I am confused about why this is the right fix,
> but it's clearly equivalent to what was here before,
> and if it works, I'm happy to postpone understanding
> until the next time Apple changes something.

I started getting into debugging the root cause, but realized all I
was trying to solve is the case where you want to build darwin/amd64
tools for a darwin/386 target, which is such a tiny audience (if
anyone!) that it hardly seems worth it.

This change will come to bite us if we want to support darwin/arm, but
it's minor enough that I'm willing to live with that.