The tests of the google-benchmark package fail to build on systems
that do not have an executbale named python in the path. Systems like
CentOS only provide python3 nowadays.
This just disables the tests as we don't need them.
Differential D151917
Disable tests for google-benchmark library build MatzeB on Jun 1 2023, 11:31 AM. Authored by
Details The tests of the google-benchmark package fail to build on systems This just disables the tests as we don't need them.
Diff Detail
Event TimelineComment Actions Could we detect the CentOS case instead of always disabling them - iirc @lebedev.ri pointed out that there is some benefit to having them on. Comment Actions FWIW: I tried fixing things upstream in https://github.com/google/benchmark/pull/1496 but that got reverted in the meantime and I don't have the time to keep pushing on this until all the corner cases work... Comment Actions Looking back at upstream it seems part of my change is still around so updating may help. That said: This change here simplifies our build and allows us to drop those odd gmock and gtest dependencies. So I still would rather use this cleanup here! Is there really any reason to run google-benchmark tests in the context of llvm-test-suite? |