rustc --test should not check linking when using --emit=metadata or -Z no-trans

[Wilfred]

Suppose I have a library configured with panic = "abort":

$ cargo init --lib foo
$ cd foo
$ nano Cargo.toml
$ more Cargo.toml
[package]
name = "foo"
version = "0.1.0"
authors = ["Wilfred Hughes <[email protected]>"]

[dependencies]

[profile.dev]
panic = "abort"

I'm unable to build with with --test:

$ cargo rustc -- --test
   Compiling foo v0.1.0 (file:///home/wilfred/tmp/foo)
error: the linked panic runtime `panic_unwind` is not compiled with this crate's panic strategy `abort`

error: aborting due to previous error

error: Could not compile `foo`.

To learn more, run the command again with --verbose.

As discussed in rust-lang/cargo#3166 (comment) this is because the test harness needs unwinding on panic.

However, I get the same error with -Z no-trans:

$ cargo rustc -- -Z no-trans --test 
   Compiling foo v0.1.0 (file:///home/wilfred/tmp/foo)
error: the linked panic runtime `panic_unwind` is not compiled with this crate's panic strategy `abort`

error: aborting due to previous error

error: Could not compile `foo`.

To learn more, run the command again with --verbose.

This causes problems in editor integration like flycheck/flycheck-rust#47 . We want to pass --test as otherwise we don't get syntax checking on test files. However, we get the above error for projects with abort-on-panic.

Could we teach --test to ignore the runtime when we pass -Z no-trans?

[steveklabnik]

-Z no-trans is on its way out, now that we have the meatdata-only crate type, right?

[nagisa]

Right; -Z no-trans will likely become an unusable actually unstable flag even before that.

[Wilfred]

Aha, great to see this stabilising. My understanding from #38666 is that --crate-type=metadata has had a short life (and had issues distinguishing between libraries and applications) and the correct version is now --emit=metadata.

We'll certainly move to --emit=metadata once it lands on stable Rust.

Note that the same problem exists with --emit=metadata though:

$ rustc --version
rustc 1.17.0-nightly (306035c21 2017-02-18)
$ cargo rustc -- --emit=metadata --test
   Compiling abortexample v0.1.0 (file:///home/wilfred/tmp/abortexample)
error: the linked panic runtime `panic_unwind` is not compiled with this crate's panic strategy `abort`

error: aborting due to previous error

error: Could not compile `abortexample`.

To learn more, run the command again with --verbose.

[Mark-Simulacrum]

As far as I can tell this works today, so I'm going to close. rustc --emit=metadata -Cpanic=abort --test test.rs works on:

#[test]
fn t() {
    panic!("oh no!");
}

[Wilfred]

@Mark-Simulacrum I don't think your test is equivalent. The example from the first comment still fails on current Rust. Could you please reopen?

[ehuss]

The intended way to do this with cargo is to use cargo check --profile=test. That forces panic to be unwind in order to be compatible with libtest.

Additionally, once #65710 lands, there is a new flag cargo check --profile=test -Zpanic-abort-tests where it will successfully build with abort.