search

kennytm / cargo-kcov

Cargo subcommand to run kcov to get coverage report on Linux
127 stars 19 forks source link

Please upgrade to kcov v34 to fix new binutils #19

Closed FauxFaux closed 7 years ago

FauxFaux commented 7 years ago

kcov v33, which cargo-kcov attempts to install, does not compile with new binutils, e.g. on Ubuntu+1: https://github.com/SimonKagstrom/kcov/issues/215

The error is:

$ cargo kcov --print-install-kcov-sh | sh
[downloading] https://github.com/SimonKagstrom/kcov/archive/v33.tar.gz
...
[ 48%] Building CXX object src/CMakeFiles/kcov.dir/output-handler.cc.o
[ 51%] Building CXX object src/CMakeFiles/kcov.dir/parsers/bfd-address-verifier.cc.o
.../kcov-33/src/parsers/bfd-address-verifier.cc: In constructor ‘BfdAddressVerifier::BfdAddressVerifier()’:
.../kcov-33/src/parsers/bfd-address-verifier.cc:26:20: error: ‘print_insn_i386’ was not declared in this scope
   m_disassembler = print_insn_i386;
                    ^~~~~~~~~~~~~~~
.../kcov-33/src/parsers/bfd-address-verifier.cc:26:20: note: suggested alternative: ‘print_insn_m32c’
   m_disassembler = print_insn_i386;
                    ^~~~~~~~~~~~~~~
                    print_insn_m32c
src/CMakeFiles/kcov.dir/build.make:390: recipe for target 'src/CMakeFiles/kcov.dir/parsers/bfd-address-verifier.cc.o' failed

Please upgrade to v34, which should have fixed this issue.