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

Issue 7546049: code review 7546049: build: skip benchmarks on OpenBSD (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by rsc
Modified:
12 years, 2 months ago
Reviewers:
bsiegert, jsing
CC:
golang-dev, bradfitz
Visibility:
Public.

Description

build: skip benchmarks on OpenBSD They are making the build die. I want to be able to see that everything else is okay.

Patch Set 1 #

Patch Set 2 : diff -r 9606c6225594 https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 9606c6225594 https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r 821f6f552e3d https://go.googlecode.com/hg/ #

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

Messages

Total messages: 5
rsc
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg/
12 years, 2 months ago (2013-03-15 15:47:34 UTC) #1
bradfitz
LGTM On Mar 15, 2013 8:47 AM, <rsc@golang.org> wrote: > Reviewers: golang-dev1, > > Message: ...
12 years, 2 months ago (2013-03-15 15:49:21 UTC) #2
rsc
*** Submitted as https://code.google.com/p/go/source/detail?r=16f65c9ab399 *** build: skip benchmarks on OpenBSD They are making the build ...
12 years, 2 months ago (2013-03-15 16:39:18 UTC) #3
bsiegert
Doesn't help: # ../test run nilptr.go : incorrect output signal: segmentation fault On Fri, Mar ...
12 years, 2 months ago (2013-03-16 10:00:09 UTC) #4
jsing
12 years, 2 months ago (2013-03-17 15:35:27 UTC) #5
On 16 March 2013 21:00, Benny Siegert <bsiegert@gmail.com> wrote:
> Doesn't help:
>
> # ../test
> run                  nilptr.go           : incorrect output
> signal: segmentation fault

I believe the new builder is running with a data seg size ulimit of
512MB, which is why these tests are failing. However, it is great when
this not only exposes a bug in the OpenBSD runtime, but bugs in the
runtimes for multiple platforms :)
Sign in to reply to this message.

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