This is an archive of the discontinued LLVM Phabricator instance.

Disable tests for google-benchmark library build
ClosedPublic

Authored by MatzeB on Jun 1 2023, 11:31 AM.

Details

Summary

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.

Diff Detail

Repository
rT test-suite

Event Timeline

MatzeB created this revision.Jun 1 2023, 11:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 11:31 AM
MatzeB requested review of this revision.Jun 1 2023, 11:31 AM
MatzeB edited the summary of this revision. (Show Details)
MatzeB edited the summary of this revision. (Show Details)

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.

MatzeB added a comment.EditedJun 1 2023, 2:22 PM

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...

MatzeB added a comment.Jun 1 2023, 2:28 PM

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?

mtrofin accepted this revision.Jun 1 2023, 3:22 PM

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?

Oh! Fun :)

LGTM, if no one objects too strongly.

This revision is now accepted and ready to land.Jun 1 2023, 3:22 PM
MatzeB updated this revision to Diff 527700.Jun 1 2023, 6:52 PM
This revision was automatically updated to reflect the committed changes.