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

Issue 72220043: code review 72220043: cmd/5c: fix arm build (Closed)

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

Description

cmd/5c: fix arm build cmd/5c/txt.c was missing from CL 72010043.

Patch Set 1 #

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

Patch Set 3 : diff -r e24ed5c8cecd https://code.google.com/p/go #

Patch Set 4 : diff -r e24ed5c8cecd https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M src/cmd/5c/txt.c View 1 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 6
dave_cheney.net
Hello rsc@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
11 years, 3 months ago (2014-03-06 23:10:58 UTC) #1
bradfitz
LGTM On Mar 6, 2014 3:11 PM, <dave@cheney.net> wrote: > Reviewers: rsc, > > Message: ...
11 years, 3 months ago (2014-03-06 23:12:28 UTC) #2
dave_cheney.net
*** Submitted as https://code.google.com/p/go/source/detail?r=7cb38c8ff249 *** cmd/5c: fix arm build cmd/5c/txt.c was missing from CL 72010043. ...
11 years, 3 months ago (2014-03-06 23:13:13 UTC) #3
gobot
This CL appears to have broken the darwin-386-cheney builder.
11 years, 3 months ago (2014-03-06 23:35:26 UTC) #4
dave_cheney.net
I believe this is an unrelated failure, # ../test # go run run.go -- index2.go ...
11 years, 3 months ago (2014-03-06 23:37:52 UTC) #5
rsc
11 years, 3 months ago (2014-03-06 23:56:15 UTC) #6
On Thu, Mar 6, 2014 at 6:37 PM, Dave Cheney <dave@cheney.net> wrote:

> I believe this is an unrelated failure,
>
> # ../test
> # go run run.go -- index2.go
>
> tmp__.go:2431: missing error "index|overflow|truncated|must be integer"
> Unmatched Errors:
> tmp__.go:2431: invalid slice index cni32 (index must be non-negative)
>
> FAIL	index2.go	27.144s
> exit status 1
>
>
Yes, that is issue 7344. I have been trying to track it down for the past
two days. No luck yet.

Russ
Sign in to reply to this message.

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