-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
Tarpaulin 0.18.0-alpha1 fails after a cargo build
#736
Comments
So there was a cargo PR about this that was meant to fix it. I tried to use the latest nightly for your project with the latest nightly compiler just to make sure I was using a version of cargo with the fix in and instead I encountered ICE rust-lang/rust#84225 . Tomorrow is the start of my weekend so I'll set aside some time to more thoroughly diff 0.16.0 and 0.18.0-alpha1 to try and see what changed and also try it out on some other projects to get to the bottom of it. |
@xd009642, txs for looking into this. This evening, I'll try to simplify the code. |
I've simplified the code its on my tarpaulin-requires-cargo-clean branch. main.rs is as simple as I could get:
tests/cli.rs is empty
In Cargo.toml I've removed the
rust-toolchain is still stable
And tarpaulin is still 0.18.0-alpha1
And running
But if I comment out
It "works":
I hope this makes identifing the problem easier. Let me know if there is anything you'd like me to do! |
So it looks like this may have been indirectly caused by the same issue being fixed by rust-lang/rust#84226 . I'll change tarpaulin to clean by default to solve it and then remove that option once this makes it into stable |
So 0.18.0-alpha2 is now released that fixes the issue until that rust PR makes it into the 1.53.0 stable, checked it and it works as expected 👍 |
Txs for adding --skip-clean! |
Describe the bug
When using Tarpaulin 0.18.0-alpha1 installed from a clone of this project:
I get the following error when I compile winksaville/binance-auto-sell:
If instead I use the release version, 0.16.0:
It completes successfully:
To Reproduce
Here is the command line and full output. Notice that this happens after a
cargo build
:Work-around
If I do a
cargo clean
before thecargo tarpaulin
it works:And if I add the
--follow-exec
parameter I see the expected 100.00% coverage:Expected behavior
The version 0.18.0-alpha1 should behave the same as 0.16.0.
System information
Arch Linux:
output of
rustup show
:My rust-toolchain file:
The text was updated successfully, but these errors were encountered: