search

launchdarkly / cpp-sdks

C++ Client/Server SDKs
Other
5 stars 2 forks source link

ci: upgrade googletest #381

Closed cwaldren-ld closed 4 months ago

cwaldren-ld commented 4 months ago

This upgrades googletest to a much more recent commit. The intention is to see if it helps with the DEADLYSIGNAL issue that has been plaguing this repo's CI for the last couple weeks.

Underlying theory is that possibly there is some incompatibilitiy between the old googletest and gcc.

shortcut-integration[bot] commented 4 months ago

This pull request has been linked to Shortcut Story #236513: Investigate DEADLYSIGNAL in CI.

cwaldren-ld commented 4 months ago

So, looks like there was one failure where the runner ran for 1+ hour out of 7 runs in the build-test-sse job (I've been using that one to trigger the DEADLYSIGNAL.)

This could be a total coincidence, but it does seem like a better failure rate than 1 out of 2 or 3.