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

Temporarily disable compiletest testing in CI #1512

Merged
merged 1 commit into from
Apr 23, 2019
Merged

Temporarily disable compiletest testing in CI #1512

merged 1 commit into from
Apr 23, 2019

Conversation

dtolnay
Copy link
Member

@dtolnay dtolnay commented Apr 23, 2019

The nightly compiler just added a dependency on serde in rust-lang/rust#60053, so libserde now ends up in the sysroot, breaking crate resolution inside of compiletest. We will need to figure out how else to run these tests.

error[E0464]: multiple matching crates for `serde`
  --> $DIR/wrong_ser.rs:9:10
   |
 9 | #[derive(Serialize)]
   |          ^^^^^^^^^
   |
   = note: candidates:
           crate `serde`: /rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/lib/libserde-2b75907288aa7c40.rlib
           crate `serde`: /serde/test_suite/deps/target/debug/deps/libserde-33e0a319242344ce.rlib

The nightly compiler just added a dependency on serde so libserde ends
up in the sysroot, breaking crate resolution inside of compiletest. We
will need to figure out how else to run these tests.

    error[E0464]: multiple matching crates for `serde`
      --> $DIR/wrong_ser.rs:9:10
       |
     9 | #[derive(Serialize)]
       |          ^^^^^^^^^
       |
       = note: candidates:
               crate `serde`: /rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/lib/libserde-2b75907288aa7c40.rlib
               crate `serde`: /serde/test_suite/deps/target/debug/deps/libserde-33e0a319242344ce.rlib
@dtolnay
Copy link
Member Author

dtolnay commented Apr 23, 2019

I filed #1513 to follow up.

@dtolnay dtolnay merged commit 1e6d3ff into serde-rs:master Apr 23, 2019
@dtolnay dtolnay deleted the off branch April 23, 2019 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant