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

update Miri #66925

Merged
merged 1 commit into from
Dec 4, 2019
Merged

update Miri #66925

merged 1 commit into from
Dec 4, 2019

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Dec 1, 2019

Fixes #66862

r? @ghost

@RalfJung
Copy link
Member Author

RalfJung commented Dec 1, 2019

Submodule-only, fix-toolstate

@bors r+ p=1
Cc @oli-obk

@bors
Copy link
Contributor

bors commented Dec 1, 2019

📌 Commit 209299c96e0e29476c93ed605cf619e374034951 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Dec 1, 2019

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 1, 2019
@bors
Copy link
Contributor

bors commented Dec 2, 2019

⌛ Testing commit 209299c96e0e29476c93ed605cf619e374034951 with merge cbbc6fc3308eba896b3563418ef9f8cab25947e5...

@RalfJung
Copy link
Member Author

RalfJung commented Dec 2, 2019

Conflicting PRs have landed.

@bors r- retry

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 2, 2019
@RalfJung RalfJung added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 2, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Dec 2, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Dec 2, 2019

📌 Commit d9b7ba494362f2133cd6583722538b791b3c7eab has been approved by RalfJung

@bors bors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 2, 2019
@rust-highfive
Copy link
Collaborator

The job x86_64-gnu-tools of your PR failed (pretty log, 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.
2019-12-02T18:56:00.4739634Z ##[command]git remote add origin https://github.com/rust-lang/rust
2019-12-02T18:56:00.4754793Z ##[command]git config gc.auto 0
2019-12-02T18:56:00.4757183Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2019-12-02T18:56:00.4759924Z ##[command]git config --get-all http.proxy
2019-12-02T18:56:00.4763065Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/66925/merge:refs/remotes/pull/66925/merge
---
2019-12-02T20:27:38.2707006Z error: The server responded with 404 Not Found for "https://forge.rust-lang.org/rustc-bug-fix-procedure.html"
2019-12-02T20:27:38.2707833Z 
2019-12-02T20:27:38.2708285Z     ┌── compiler-team.md:40:3 ───
2019-12-02T20:27:38.2708626Z     │
2019-12-02T20:27:38.2709001Z  40 │   [aim to give warnings first][procedure]).
2019-12-02T20:27:38.2709658Z     │
2019-12-02T20:27:38.2709875Z 
2019-12-02T20:27:38.2710257Z error: The server responded with 404 Not Found for "https://forge.rust-lang.org/platform-support.html"
2019-12-02T20:27:38.2710416Z 
2019-12-02T20:27:38.2710416Z 
2019-12-02T20:27:38.2710755Z      ┌── tests/intro.md:116:8 ───
2019-12-02T20:27:38.2711259Z      │
2019-12-02T20:27:38.2711858Z  116 │ Rust's [platform tiers]).
2019-12-02T20:27:38.2712567Z      │
2019-12-02T20:27:38.2712723Z 
2019-12-02T20:27:38.2713184Z error: The server responded with 404 Not Found for "https://doc.rust-lang.org/nightly/nightly-rustc/rustc/middle/expr_use_visitor/struct.ExprUseVisitor.html"
2019-12-02T20:27:38.2713376Z 
---
2019-12-02T20:57:21.4702865Z 
2019-12-02T20:57:21.4703093Z If you do intend to update 'miri', please check the error messages above and
2019-12-02T20:57:21.4703137Z commit another update.
2019-12-02T20:57:21.4703160Z 
2019-12-02T20:57:21.4704132Z If you do NOT intend to update 'miri', please ensure you did not accidentally
2019-12-02T20:57:21.4708307Z change the submodule at 'src/tools/miri'. You may ask your reviewer for the
2019-12-02T20:57:21.4708370Z proper steps.
2019-12-02T20:57:21.4722712Z   local time: Mon Dec  2 20:57:21 UTC 2019
2019-12-02T20:57:21.5225690Z   network time: Mon, 02 Dec 2019 20:57:21 GMT
2019-12-02T20:57:21.5231052Z == end clock drift check ==
2019-12-02T20:57:22.6936004Z 
2019-12-02T20:57:22.6936004Z 
2019-12-02T20:57:22.7042209Z ##[error]Bash exited with code '3'.
2019-12-02T20:57:22.7073099Z ##[section]Starting: Checkout
2019-12-02T20:57:22.7075303Z ==============================================================================
2019-12-02T20:57:22.7075354Z Task         : Get sources
2019-12-02T20:57:22.7075398Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

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)

@bors
Copy link
Contributor

bors commented Dec 2, 2019

⌛ Testing commit d9b7ba494362f2133cd6583722538b791b3c7eab with merge 28fcd1038cb3b3ad60f2b961410e40d76f910b09...

@RalfJung
Copy link
Member Author

RalfJung commented Dec 2, 2019

Looks like something landed that broke Miri (agian...)

2019-12-02T20:57:19.8288172Z error[E0308]: try expression alternatives have incompatible types
2019-12-02T20:57:19.8289330Z --> src/tools/miri/src/shims/mod.rs:52:17
2019-12-02T20:57:19.8289702Z |
2019-12-02T20:57:19.8290135Z 52 | Ok(Some(this.load_mir(instance.def, None)?))
2019-12-02T20:57:19.8290620Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `&rustc::mir::Body<'_>`, found struct `rustc::mir::ReadOnlyBodyCache`
2019-12-02T20:57:19.8290800Z
2019-12-02T20:57:19.9292737Z error[E0308]: try expression alternatives have incompatible types
2019-12-02T20:57:19.9293073Z --> src/tools/miri/src/shims/foreign_items.rs:144:32
2019-12-02T20:57:19.9293295Z |
2019-12-02T20:57:19.9293549Z 144 | return Ok(Some(this.load_mir(start_panic_instance.def, None)?));
2019-12-02T20:57:19.9295202Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `&rustc::mir::Body<'_>`, found struct `rustc::mir::ReadOnlyBodyCache`
2019-12-02T20:57:19.9295256Z
2019-12-02T20:57:19.9370379Z error[E0308]: try expression alternatives have incompatible types
2019-12-02T20:57:19.9370643Z --> src/tools/miri/src/shims/foreign_items.rs:151:32
2019-12-02T20:57:19.9370844Z |
2019-12-02T20:57:19.9371125Z 151 | return Ok(Some(this.load_mir(panic_impl_instance.def, None)?));
2019-12-02T20:57:19.9371471Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `&rustc::mir::Body<'_>`, found struct `rustc::mir::ReadOnlyBodyCache`
2019-12-02T20:57:19.9371528Z
2019-12-02T20:57:21.0463739Z error[E0308]: mismatched types
2019-12-02T20:57:21.0464067Z --> src/tools/miri/src/helpers.rs:142:13
2019-12-02T20:57:21.0464484Z |
2019-12-02T20:57:21.0464717Z 142 | mir,
2019-12-02T20:57:21.0464943Z | ^^^
2019-12-02T20:57:21.0465179Z | |
2019-12-02T20:57:21.0465513Z | expected `&rustc::mir::Body<'_>`, found struct `rustc::mir::ReadOnlyBodyCache`
2019-12-02T20:57:21.0465772Z | help: consider borrowing here: `&mir`

@bors r- retry

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 2, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Dec 3, 2019

@pietroalbini bors seems very confused with its tags? This is the second time recently that I did @bors r- and the tag remains "S-waiting-on-bors". GH also seems confused

added S-waiting-on-author S-waiting-on-bors and removed S-waiting-on-bors S-waiting-on-author

Not sure how it could even remove a tag that wasn't there... or add one that already was there...

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 3, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Dec 3, 2019

Hm, doing it a second time seems to work? Or a fix landed in the mean time.

@RalfJung
Copy link
Member Author

RalfJung commented Dec 3, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Dec 3, 2019

📌 Commit 48689f00145ef50dedfbc7e9e676c1e48da59ed9 has been approved by RalfJung

@bors bors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 3, 2019
@bors
Copy link
Contributor

bors commented Dec 3, 2019

⌛ Testing commit 48689f00145ef50dedfbc7e9e676c1e48da59ed9 with merge 74cb058cb5c86e7d7729d020c3d01d6e461b01ea...

@RalfJung
Copy link
Member Author

RalfJung commented Dec 3, 2019

#66148 changed a Miri test output.

@bors r- retry

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 3, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Dec 3, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Dec 3, 2019

📌 Commit 6b7f634 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Dec 3, 2019

⌛ Testing commit 6b7f634 with merge 2f04472...

bors added a commit that referenced this pull request Dec 3, 2019
@bors
Copy link
Contributor

bors commented Dec 4, 2019

☀️ Test successful - checks-azure
Approved by: RalfJung
Pushing 2f04472 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 4, 2019
@bors bors merged commit 6b7f634 into rust-lang:master Dec 4, 2019
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #66925!

Tested on commit 2f04472.
Direct link to PR: #66925

🎉 miri on windows: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).
🎉 miri on linux: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Dec 4, 2019
Tested on commit rust-lang/rust@2f04472.
Direct link to PR: <rust-lang/rust#66925>

🎉 miri on windows: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).
🎉 miri on linux: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).
@RalfJung RalfJung deleted the miri branch December 4, 2019 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

miri no longer builds after rust-lang/rust#66547
3 participants