Skip to content

[CI] Update GoogleTest#11162

Merged
Mousius merged 2 commits intoapache:mainfrom
Mousius:better-googletest
May 3, 2022
Merged

[CI] Update GoogleTest#11162
Mousius merged 2 commits intoapache:mainfrom
Mousius:better-googletest

Conversation

@Mousius
Copy link
Copy Markdown
Member

@Mousius Mousius commented Apr 28, 2022

This updates GoogleTest across all our images which also has the side
effect of using the same version for any host OS.

Rather than updating to a fixed version, I've followed the best practice
advertised by GoogleTest itself which is the Live-at-Head philosophy:
https://github.com/google/googletest#live-at-head

Closes #11002

cc @areusch @driazati

@Mousius Mousius force-pushed the better-googletest branch from 22bfcbf to e5dd3bc Compare April 28, 2022 14:43
@github-actions github-actions Bot requested review from areusch and driazati April 28, 2022 14:45
@Mousius Mousius force-pushed the better-googletest branch from e5dd3bc to 17a7be0 Compare April 28, 2022 15:39
Copy link
Copy Markdown
Contributor

@areusch areusch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @Mousius -- just to double check, have you rebuilt image locally and verified you can still run ninja cpptest from build?

@Mousius
Copy link
Copy Markdown
Member Author

Mousius commented Apr 28, 2022

@areusch I have, and I get the same number of seg faults, which isn't very encouraging for my local environment - need to see what happens in CI here

Mousius added 2 commits May 3, 2022 10:04
This updates GoogleTest across all our images which also has the side
effect of using the same version for any host OS.

Rather than updating to a fixed version, I've followed the best practice
advertised by GoogleTest itself which is the Live-at-Head philosophy:
https://github.com/google/googletest#live-at-head

Closes apache#11002
This is needed to make many public HTTPS requests and it's likely installed as a
side effect right now.
Copy link
Copy Markdown
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @Mousius @areusch!

@Mousius
Copy link
Copy Markdown
Member Author

Mousius commented May 3, 2022

Update on the above, I found it was USE_RELAY_DEBUG which was causing the C++ tests to seg fault for me, they all passed with this now 😸

@Mousius Mousius merged commit 91d351c into apache:main May 3, 2022
@Mousius Mousius deleted the better-googletest branch May 3, 2022 15:46
Mousius added a commit to Mousius/tvm that referenced this pull request May 4, 2022
Looks like I missed this one in apache#11162 due to missing the cpptest call
in the Jenkinsfile
Mousius added a commit that referenced this pull request May 4, 2022
Looks like I missed this one in #11162 due to missing the cpptest call
in the Jenkinsfile
shtinsa pushed a commit to Deelvin/tvm that referenced this pull request May 17, 2022
This updates GoogleTest across all our images which also has the side
effect of using the same version for any host OS.

Rather than updating to a fixed version, I've followed the best practice
advertised by GoogleTest itself which is the Live-at-Head philosophy:
https://github.com/google/googletest#live-at-head

Closes apache#11002
shtinsa pushed a commit to Deelvin/tvm that referenced this pull request May 17, 2022
Looks like I missed this one in apache#11162 due to missing the cpptest call
in the Jenkinsfile
SebastianBoblest pushed a commit to SebastianBoblest/tvm that referenced this pull request May 27, 2022
This updates GoogleTest across all our images which also has the side
effect of using the same version for any host OS.

Rather than updating to a fixed version, I've followed the best practice
advertised by GoogleTest itself which is the Live-at-Head philosophy:
https://github.com/google/googletest#live-at-head

Closes apache#11002
SebastianBoblest pushed a commit to SebastianBoblest/tvm that referenced this pull request May 27, 2022
Looks like I missed this one in apache#11162 due to missing the cpptest call
in the Jenkinsfile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tracking Issue] Update to googletest 1.11.0

3 participants