Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update google_benchmark to v1.5.3 #16

Merged
merged 1 commit into from
May 25, 2021
Merged

Update google_benchmark to v1.5.3 #16

merged 1 commit into from
May 25, 2021

Conversation

homalozoa
Copy link
Contributor

  1. Change google_benchmark version from v1.5.2 to v1.5.3. Because v1.5.2
    can not build with GCC 11
  2. Removed shrink-tz-offset-size.patch because of this patch was merged in
    google-benchmark repo.

Signed-off-by: Homalozoa nx.tardis@gmail.com

1. Change google_benchmark version from v1.5.2 to v1.5.3. Because v1.5.2
   can not build with GCC 11
2. Removed shrink-tz-offset-size.patch because of this patch was merged in
google-benchmark repo.

Signed-off-by: Homalozoa <nx.tardis@gmail.com>
@clalancette
Copy link
Contributor

Let's run CI with this and see how it looks:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor

I think that the CMake warnings on aarch64 and Windows are expected for the performance benchmarks, but I'm not sure. @cottsay Could you take a look at this PR and the CI run and see if it makes sense to you?

@cottsay
Copy link
Contributor

cottsay commented May 24, 2021

Could you take a look at this PR and the CI run and see if it makes sense to you?

Yes, those warnings are expected with AMENT_RUN_PERFORMANCE_TESTS. This change looks good to me.

@clalancette
Copy link
Contributor

Yes, those warnings are expected with AMENT_RUN_PERFORMANCE_TESTS. This change looks good to me.

Thanks, appreciated. Then I'll go ahead and merge this.

@clalancette clalancette merged commit 9416ce9 into ament:main May 25, 2021
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.

None yet

3 participants