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

Coerce fields to the expected field type #42807

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Jun 21, 2017

Fully fixes #31260.

This needs a crater run. I was supposed to do this last month but it slipped. Let's get this done.

Fully fixes rust-lang#31260.

This needs a crater run.
@rust-highfive
Copy link
Collaborator

r? @pnkfelix

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

@arielb1 arielb1 added the S-waiting-on-crater Status: Waiting on a crater run to be completed. label Jun 21, 2017
@eddyb
Copy link
Member

eddyb commented Jun 22, 2017

Started crater run.

@eddyb
Copy link
Member

eddyb commented Jun 23, 2017

Crater report shows 0 regressions (ignoring network errors).

@eddyb
Copy link
Member

eddyb commented Jun 23, 2017

cc @rust-lang/compiler

@arielb1 arielb1 changed the title [NEEDS CRATER] coerce fields to the expected field type Coerce fields to the expected field type Jun 23, 2017
@shepmaster shepmaster added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Jun 23, 2017
@arielb1 arielb1 added I-nominated T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 28, 2017
@nikomatsakis
Copy link
Contributor

Discussed in the @rust-lang/compiler meeting briefly. If no regressions, seems like we should do this.

@eddyb
Copy link
Member

eddyb commented Jun 29, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Jun 29, 2017

📌 Commit 7769c9a has been approved by eddyb

@bors
Copy link
Contributor

bors commented Jun 29, 2017

⌛ Testing commit 7769c9a with merge 7a087743874c8f3bd906ff84806361af2b31cd6b...

@bors
Copy link
Contributor

bors commented Jun 29, 2017

💔 Test failed - status-travis

@Mark-Simulacrum
Copy link
Member

I suspect this is a spurious failure; it looks like the test will exit after 30 seconds via process::exit(1) -- which should probably be 0, really, I think.

[01:03:36] failures:
[01:03:36]
[01:03:36] ---- [run-pass] run-pass/tcp-stress.rs stdout ----
[01:03:36]
[01:03:36] error: test run failed!
[01:03:36] status: exit code: 1
[01:03:36] command: /checkout/obj/build/x86_64-unknown-linux-gnu/test/run-pass/tcp-stress.stage2-x86_64-unknown-linux-gnu
[01:03:36] stdout:
[01:03:36] ------------------------------------------
[01:03:36]
[01:03:36] ------------------------------------------
[01:03:36] stderr:
[01:03:36] ------------------------------------------
[01:03:36]
[01:03:36] ------------------------------------------
[01:03:36]
[01:03:36] thread '[run-pass] run-pass/tcp-stress.rs' panicked at 'explicit panic', /checkout/src/tools/compiletest/src/runtest.rs:2479
[01:03:36] note: Run with `RUST_BACKTRACE=1` for a backtrace.

@Mark-Simulacrum
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Jun 30, 2017

⌛ Testing commit 7769c9a with merge 8b35998e0a2a3967eb0bc33fc37256692a93b6b2...

@bors
Copy link
Contributor

bors commented Jun 30, 2017

💔 Test failed - status-appveyor

@Mark-Simulacrum
Copy link
Member

@bors retry

#41006

[01:46:36] error: failed to remove: C:\projects\rust\build\i686-pc-windows-msvc\stage1-rustc\i686-pc-windows-msvc\release\rustc_driver.dll
[01:46:36]
[01:46:36] Caused by:
[01:46:36]   Access is denied. (os error 5)
[01:46:36] Build failed, waiting for other jobs to finish...
[01:46:40] error: build failed

@bors
Copy link
Contributor

bors commented Jun 30, 2017

⌛ Testing commit 7769c9a with merge e72580c...

bors added a commit that referenced this pull request Jun 30, 2017
Coerce fields to the expected field type

Fully fixes #31260.

This needs a crater run. I was supposed to do this last month but it slipped. Let's get this done.
@bors
Copy link
Contributor

bors commented Jun 30, 2017

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant 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.

Coercion sites are not consistent between struct tuples and structs
8 participants