search

mesonbuild / meson

The Meson Build System
http://mesonbuild.com
Apache License 2.0
5.51k stars 1.6k forks source link

coverage dependencies: genhtml should not be needed for llvm-cov #11156

Open sycc90 opened 1 year ago

sycc90 commented 1 year ago

This is not a bug, just a question. Why does meson require genhtml when using llvm-cov? llvm-cov already allows you to generate html reports directly.

tristan957 commented 1 year ago

@eli-schwartz closeable. This can be moved to a GitHub discussion

eli-schwartz commented 1 year ago

Good question. It seems like this may not actually be necessary, we just don't distinguish between llvm-cov and lcov? Dating back to the addition of llvm-cov support in commit a198e5d191820fda9142d248cd5d134e5f2a5b93.

So actually I think this is probably a bug, or at least a future enhancement?

tristan957 commented 1 year ago

Perhaps it would be good to change the title of the issue then