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

[beta] std: Back out backtrace pruning logic on beta #40684

Merged
merged 3 commits into from
Mar 27, 2017

Conversation

alexcrichton
Copy link
Member

It was discovered #40264 that this backtrace pruning logic is a little too
aggressive, so while we figure how out to handle #40264 this commit backs out
the changes on beta to prune frames. Note that other cosmetic changes, such as
better path printing and such remain.

It was discovered rust-lang#40264 that this backtrace pruning logic is a little too
aggressive, so while we figure how out to handle rust-lang#40264 this commit backs out
the changes on beta to prune frames. Note that other cosmetic changes, such as
better path printing and such remain.
@rust-highfive
Copy link
Collaborator

r? @brson

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

@rust-highfive
Copy link
Collaborator

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@alexcrichton alexcrichton added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Mar 20, 2017
@alexcrichton alexcrichton mentioned this pull request Mar 21, 2017
5 tasks
@brson
Copy link
Contributor

brson commented Mar 23, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Mar 23, 2017

📌 Commit 810fd29 has been approved by brson

@bors
Copy link
Contributor

bors commented Mar 25, 2017

⌛ Testing commit 810fd29 with merge 8cf8338...

@bors
Copy link
Contributor

bors commented Mar 25, 2017

💔 Test failed - status-travis

@arielb1
Copy link
Contributor

arielb1 commented Mar 25, 2017

arielb1 and others added 2 commits March 27, 2017 06:53
The emsdk-portable .tar.gz now extracts to emsdk-portable instead of
emsdk_portable. Handle that.
@alexcrichton
Copy link
Member Author

@bors: r=brson

@bors
Copy link
Contributor

bors commented Mar 27, 2017

📌 Commit 7573111 has been approved by brson

@bors
Copy link
Contributor

bors commented Mar 27, 2017

⌛ Testing commit 7573111 with merge 646873b...

bors added a commit that referenced this pull request Mar 27, 2017
[beta] std: Back out backtrace pruning logic on beta

It was discovered #40264 that this backtrace pruning logic is a little too
aggressive, so while we figure how out to handle #40264 this commit backs out
the changes on beta to prune frames. Note that other cosmetic changes, such as
better path printing and such remain.
@bors
Copy link
Contributor

bors commented Mar 27, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 646873b to beta...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants