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.

Sign up for GitHub

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

Jump to bottom

Coverage found the following executables: [] #54

Open
asomers opened this issue Aug 4, 2023 · 1 comment
Open

Coverage found the following executables: [] #54

asomers opened this issue Aug 4, 2023 · 1 comment

Comments

@asomers
Copy link

asomers commented Aug 4, 2023

cargo-kcov no longer works with recent versions of cargo. It tries to invoke kcov without any executable, which kcov treats as a syntax error. The last Cargo version that I know of as working is 1.68.0, and the first broken version is 1.73.0. I haven't tried to bisect it further.

> cargo kcov --verbose
       Clean file:///usr/home/somers/src/rust/mio-aio#0.7.0

       Build test executables
    Coverage found the following executables: []
     Running "kcov" "--merge" "--exclude-pattern=/home/somers/.cargo" "--verify" "/usr/home/somers/src/rust/mio-aio/target/cov"
kcov: warning: kcov: WARNING: kcov has been built without libbfd-dev (or
kcov: binutils-dev), so the --verify option will not do anything.

Usage: kcov [OPTIONS] out-dir in-file [args...]
...
asomers added a commit to asomers/futures-locks that referenced this issue Aug 5, 2023
@asomers
Switch CI's coverage checker from kcov to cargo-llvm-cov
036b0ac 
Because cargo-kcov is broken and unmaintained.
kennytm/cargo-kcov#54
@asomers asomers mentioned this issue Aug 5, 2023
Merged
@mariusknaust
Copy link

It actually stops working with 1.71.0, hence 1.70.0 is the last compatible version.

roypat added a commit to roypat/rust-vmm-container that referenced this issue Aug 30, 2023
@roypat
Replace cargo-kcov with grcov
fa035f5 
cargo-kcov is no longer working as of rust toolchain version 1.71.0, as
it can no longer discover test executables. This is due to cargo
changing its verbose stdout printing to include the qualified path to
rustc, which causes cargo-kcov's string matching to fail.

See also kennytm/cargo-kcov#54

cargo-kcov has had its last commit in 2019, so replace it with
cargo-llvm-cov, which is actively maintained, and written for rust.

Signed-off-by: Patrick Roy <[email protected]>
roypat added a commit to roypat/rust-vmm-container that referenced this issue Aug 30, 2023
@roypat
Replace cargo-kcov with llvm-cov
0c0b606 
cargo-kcov is no longer working as of rust toolchain version 1.71.0, as
it can no longer discover test executables. This is due to cargo
changing its verbose stdout printing to include the qualified path to
rustc, which causes cargo-kcov's string matching to fail.

See also kennytm/cargo-kcov#54

cargo-kcov has had its last commit in 2019, so replace it with
cargo-llvm-cov, which is actively maintained, and written for rust.

Signed-off-by: Patrick Roy <[email protected]>
@roypat roypat mentioned this issue Aug 30, 2023
Merged
4 tasks
stefano-garzarella pushed a commit to rust-vmm/rust-vmm-container that referenced this issue Aug 30, 2023
@roypat @stefano-garzarella
Replace cargo-kcov with llvm-cov
9347c52 
cargo-kcov is no longer working as of rust toolchain version 1.71.0, as
it can no longer discover test executables. This is due to cargo
changing its verbose stdout printing to include the qualified path to
rustc, which causes cargo-kcov's string matching to fail.

See also kennytm/cargo-kcov#54

cargo-kcov has had its last commit in 2019, so replace it with
cargo-llvm-cov, which is actively maintained, and written for rust.

Signed-off-by: Patrick Roy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@asomers @mariusknaust