Error when specifying LLVM tools that are incompatible with LLVM version of rustc (Homebrew LLVM 17.0.6 vs rustc 1.78.0-beta -- LLVM 18.1.2) about cargo-llvm-cov HOT 6 CLOSED

The behavior changes included in this release only change a case that previously resulted in an error (#360), so I guess the problem is most likely a mismatch between the LLVM version of LLVM tools and the LLVM version of rustc.

IIUC, CI using rustc beta (LLVM 18.1.2) and Homebrew LLVM 17.0.6. (When 0.6.8 was tested they were probably both LLVM 17.)

Considering that it appears that CI uses rustup to install rustc, a reasonable solution to this would be to remove llvm-tools-related environment variables and use llvm-tools-preview, which is distributed via rustup. Alternatively, using stable rustc instead of beta for testing would also work. The latter does not address the underlying problem, though.

By the way, the title of this issue is "build failure", but it appears that it is the test that is actually failing.

from cargo-llvm-cov.

(Unrelated to this issue, but it appears that homebrew-core uses the crate generated by cargo new --bin to test. Since the crate generated that way does not contain any test, I think it would be better to use the crate generated by cargo new --lib.)

from cargo-llvm-cov.

IIUC, CI using rustc beta (LLVM 18.1.2) and Homebrew LLVM 17.0.6. (When 0.6.8 was tested they were probably both LLVM 17.)

I was thinking the same. Thanks for confirming it.

By the way, the title of this issue is "build failure", but it appears that it is the test that is actually failing.

oh, that is mainly it is because I think it is build related as it is a version bump (which caused the test failure)

from cargo-llvm-cov.

(Unrelated to this issue, but it appears that homebrew-core uses the crate generated by cargo new --bin to test. Since the crate generated that way does not contain any test, I think it would be better to use the crate generated by cargo new --lib.)

that makes sense to me, do you want to take on a PR to do the update :)

from cargo-llvm-cov.

Filed Homebrew/homebrew-core#168319 to fix.

from cargo-llvm-cov.

The problem on the Homebrew side has been fixed in Homebrew/homebrew-core#168319 & cargo-llvm-cov documentation has also been updated and a note on compatibility when setting LLVM_* env vars has been duplicated in an easy-to-find place (fa229e3).

from cargo-llvm-cov.