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

Bump to 1.6.0 #30

Merged
merged 4 commits into from
Sep 12, 2019
Merged

Bump to 1.6.0 #30

merged 4 commits into from
Sep 12, 2019

Conversation

fitzgen
Copy link
Collaborator

@fitzgen fitzgen commented Sep 9, 2019

No description provided.

@fitzgen
Copy link
Collaborator Author

fitzgen commented Sep 9, 2019

It seems like installing / running miri is failing in CI, but when I do it locally, it works out fine. Any idea what's going on here @RalfJung? Should CI be doing cargo clean before running miri tests?

@RalfJung
Copy link
Contributor

RalfJung commented Sep 9, 2019

This is odd, I have not seen that failure before. It does install the latest xargo, I am not sure what else there might be that could be causing trouble.

Somehow the rustc-std-workspace-std patch does not seem to kick in?

@RalfJung
Copy link
Contributor

RalfJung commented Sep 9, 2019

@alexcrichton something odd seems to happen, there's a version 1.0.1 of rustc-std-workspace-std on crates.io but the rustc workspace only contains a version 1.0.0? It looks like that version mismatch makes cargo pick the crates.io-version even though xargo set up a patch to use the local version instead.

@fitzgen
Copy link
Collaborator Author

fitzgen commented Sep 9, 2019

Hm, since you don't know the fix, I'm tempted to configure CI to allow this job to fail, with the hopes that this bug eventually gets fixed upstream :-/

Do you have a better plan than that?

@RalfJung
Copy link
Contributor

RalfJung commented Sep 9, 2019

Do you have a better plan than that?

If my analysis is correct, the bug is outside of your or my control, so no, I do not have a better plan.
Hopefully @alexcrichton has an idea.

@alexcrichton
Copy link

Should be fixed in rust-lang/rust#64320

@RalfJung
Copy link
Contributor

RalfJung commented Sep 9, 2019

@alexcrichton that was quick, thanks!

@fitzgen
Copy link
Collaborator Author

fitzgen commented Sep 12, 2019

Hm ok so I retriggered these tests now that there are new nightlies out with Alex's fix, and now there are issues about backtrace:

error[E0432]: unresolved import `crate::sys_common::backtrace`

  --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:98:24

   |

98 | use crate::sys_common::backtrace::{output_filename, lock};

   |                        ^^^^^^^^^ could not find `backtrace` in `sys_common`

error[E0432]: unresolved import `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:100:5

    |

100 | use backtrace::BytesOrWideString;

    |     ^^^^^^^^^ help: a similar path exists: `crate::backtrace`

    |

    = note: `use` statements changed in Rust 2018; read more at <https://doc.rust-lang.org/edition-guide/rust-2018/module-system/path-clarity.html>

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:146:12

    |

146 |     frame: backtrace::Frame,

    |            ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:231:13

    |

231 |             backtrace::trace_unsynchronized(|frame| {

    |             ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:285:35

    |

285 |             (&capture.frames[..], backtrace::PrintFmt::Full)

    |                                   ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:287:55

    |

287 |             (&capture.frames[capture.actual_start..], backtrace::PrintFmt::Short)

    |                                                       ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:299:21

    |

299 |         let mut f = backtrace::BacktraceFmt::new(fmt, style, &mut print_path);

    |                     ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:309:54

    |

309 |                         symbol.name.as_ref().map(|b| backtrace::SymbolName::new(b)),

    |                                                      ^^^^^^^^^ use of undeclared type or module `backtrace`

error[E0433]: failed to resolve: use of undeclared type or module `backtrace`

   --> /home/travis/.rustup/toolchains/nightly-2019-09-12-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libstd/backtrace.rs:339:17

    |

339 |                 backtrace::resolve_frame_unsynchronized(&frame.frame, |symbol| {

    |                 ^^^^^^^^^ use of undeclared type or module `backtrace`

error: aborting due to 9 previous errors

At this point, I'm going to mark the miri tests as "allow fail" until this stuff gets fixed upstream.

The miri toolchain/infra is pretty flaky at the moment, unfortunately.
@fitzgen fitzgen merged commit 89c3ab8 into master Sep 12, 2019
@fitzgen fitzgen deleted the bump-to-1.6.0 branch September 12, 2019 16:08
@RalfJung
Copy link
Contributor

Yeah just saw these "backtrace" things on our CI as well. So far I have no idea what that is about.

Really the biggest problem is that rust-lang/rust#60301 is still open, which means that we ship Miri even if we know the tests are broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants