Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

travis: Turn off core dumps on OSX #39320

Merged
merged 1 commit into from
Jan 27, 2017
Merged

Conversation

alexcrichton
Copy link
Member

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.
@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@aturon
Copy link
Member

aturon commented Jan 26, 2017

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 26, 2017

📌 Commit 7095a48 has been approved by aturon

@alexcrichton
Copy link
Member Author

@bors: p=1

I think this is causing timeouts, so would love to test getting this in quickly to diagnose.

@bors
Copy link
Contributor

bors commented Jan 27, 2017

⌛ Testing commit 7095a48 with merge 62a7258...

bors added a commit that referenced this pull request Jan 27, 2017
travis: Turn off core dumps on OSX

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.
@bors
Copy link
Contributor

bors commented Jan 27, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: aturon
Pushing 62a7258 to master...

@bors bors merged commit 7095a48 into rust-lang:master Jan 27, 2017
@alexcrichton alexcrichton deleted the less-backtraces branch February 11, 2017 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants