Skip to content

Feature gate where clauses on associated type impls - #53235

Merged
bors merged 1 commit into
rust-lang:masterfrom
varkor:gat_impl_where
Aug 23, 2018
Merged

Feature gate where clauses on associated type impls#53235
bors merged 1 commit into
rust-lang:masterfrom
varkor:gat_impl_where

Conversation

@varkor

Copy link
Copy Markdown
Contributor

Fixes#52913. This doesn't address the core problem, which is tracked by #47206. However, it fixes the stable-to-stable regression: you now have to enable #![feature(generic_associated_types)] to trigger the weird behaviour.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @eddyb

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 9, 2018
@estebank

Copy link
Copy Markdown
Contributor

@bors r+
@rust-lang/compiler beta nominating due to stable-to-stable regression.

@bors

bors commented Aug 9, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 1aa6c23 has been approved by estebank

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 9, 2018
@estebankestebank added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Aug 9, 2018
@kennytmkennytm mentioned this pull request Aug 10, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Aug 11, 2018
Feature gate where clauses on associated type impls
Fixesrust-lang#52913. This doesn't address the core problem, which is tracked by rust-lang#47206. However, it fixes the stable-to-stable regression: you now have to enable `#![feature(generic_associated_types)]` to trigger the weird behaviour.
@kennytmkennytm mentioned this pull request Aug 14, 2018
@RalfJung

Copy link
Copy Markdown
Member

@bors r=estebank

Seems something is stuck?

@bors

bors commented Aug 22, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 83d5a60 has been approved by estebank

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 22, 2018
Feature gate where clauses on associated type impls
Fixesrust-lang#52913. This doesn't address the core problem, which is tracked by rust-lang#47206. However, it fixes the stable-to-stable regression: you now have to enable `#![feature(generic_associated_types)]` to trigger the weird behaviour.
@bors

bors commented Aug 23, 2018

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 83d5a60 with merge a4bf6d10c12116a1e4111c51560d5134de96d00b...

@bors

bors commented Aug 23, 2018

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@borsbors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 23, 2018
@rust-highfive

Copy link
Copy Markdown
Contributor

Your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@kennytm

Copy link
Copy Markdown
Member

@bors retry

An error occurred while generating the build script.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 23, 2018
@bors

bors commented Aug 23, 2018

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 83d5a60 with merge c648b0b...

bors added a commit that referenced this pull request Aug 23, 2018
Feature gate where clauses on associated type impls
Fixes#52913. This doesn't address the core problem, which is tracked by #47206. However, it fixes the stable-to-stable regression: you now have to enable `#![feature(generic_associated_types)]` to trigger the weird behaviour.
@bors

bors commented Aug 23, 2018

Copy link
Copy Markdown
Collaborator

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

@bors
bors merged commit 83d5a60 into rust-lang:masterAug 23, 2018
@emilyalbiniemilyalbini added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Aug 25, 2018
@nikomatsakisnikomatsakis added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Aug 25, 2018
@nikomatsakis

Copy link
Copy Markdown
Contributor

Accepting for beta (cc @rust-lang/compiler) -- small patch, fixes stability hole.

@emilyalbiniemilyalbini mentioned this pull request Aug 25, 2018
@emilyalbiniemilyalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Aug 25, 2018
bors added a commit that referenced this pull request Aug 25, 2018
[beta] Rollup backports
Merged and approved:
* #53559: add macro check for lint
* #53509: resolve: Reject some inaccessible candidates sooner during import resolution
* #53239: rustc_codegen_llvm: Restore the closure env alloca hack for LLVM 5.
* #53235: Feature gate where clauses on associated type impls
* #53516: resolve: Continue search in outer scopes after applying derive resolution fallback
r? @ghost
@CentrilCentril added the F-generic_associated_types `#![feature(generic_associated_types)]` a.k.a. GATs label Aug 5, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-acceptedAccepted for backporting to the compiler in the beta channel.F-generic_associated_types`#![feature(generic_associated_types)]` a.k.a. GATsS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rustc doesn't output anything but exits with 101 and doesn't emit any files

10 participants

@varkor@rust-highfive@estebank@bors@RalfJung@kennytm@nikomatsakis@eddyb@Centril@emilyalbini