Skip to content

Removes Default for Box<Path>. #40539

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 17, 2017
Merged

Conversation

clarfonthey
Copy link
Contributor

@clarfonthey clarfonthey commented Mar 15, 2017

This was not supposed to be added. cc @alexcrichton

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@bors
Copy link
Collaborator

bors commented Mar 15, 2017

☔ The latest upstream changes (presumably #40009) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton alexcrichton added beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. labels Mar 15, 2017
@alexcrichton
Copy link
Member

Looks great to me, thanks! r=me with a rebase

@clarfonthey
Copy link
Contributor Author

Just rebased, should work now!

@alexcrichton
Copy link
Member

@bors: r+ p=1

(beta nominated)

@bors
Copy link
Collaborator

bors commented Mar 15, 2017

📌 Commit 5e2d592 has been approved by alexcrichton

@alexcrichton
Copy link
Member

@clarcharr according to #40550 a test may need an update as well?

@clarfonthey
Copy link
Contributor Author

Oh, whoops. Updated the PR.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Mar 15, 2017

📌 Commit 0739ece has been approved by alexcrichton

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Mar 17, 2017
…ichton

Removes Default for Box<Path>.

This was not supposed to be added. cc @alexcrichton
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Mar 17, 2017
…ichton

Removes Default for Box<Path>.

This was not supposed to be added. cc @alexcrichton
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Mar 17, 2017
…ichton

Removes Default for Box<Path>.

This was not supposed to be added. cc @alexcrichton
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Mar 17, 2017
…ichton

Removes Default for Box<Path>.

This was not supposed to be added. cc @alexcrichton
@bors
Copy link
Collaborator

bors commented Mar 17, 2017

⌛ Testing commit 0739ece with merge f77ee2f...

bors added a commit that referenced this pull request Mar 17, 2017
Removes Default for Box<Path>.

This was not supposed to be added. cc @alexcrichton
@bors
Copy link
Collaborator

bors commented Mar 17, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing f77ee2f to master...

@bors bors merged commit 0739ece into rust-lang:master Mar 17, 2017
bors added a commit that referenced this pull request Mar 17, 2017
[beta] Backports to beta

This is a backport of the following PRs to beta:

* #40507
* #40482
* #40526
* #40539
* #40538
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Mar 20, 2017
@clarfonthey clarfonthey deleted the no_default_path branch April 1, 2017 17:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants