cellml / libcellml

Repository for libCellML development.
https://libcellml.org
Apache License 2.0
17 stars 21 forks source link

LLVM coverage: don't use hints to find llvm-cov and llvm-profdata and let the user know where the report can be found #1257

Closed agarny closed 4 weeks ago

agarny commented 1 month ago

Someone might have a version of llvm-cov and llvm-profdata that they use and which is not located in /Library/Developer/CommandLineTools/usr/bin. This means that, right now, they would need to provide LLVM_BIN_DIR which is not ideal, not least since there are PREFERRED_LLVM_COV_NAMES and PREFERRED_LLVM_PROFDATA_NAMES to play with, if needed.

agarny commented 4 weeks ago

This is now done (and improved) as part of issue #1253, so closing.