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 Valgrind version in CI docker image #4759

Closed
boquan-fang opened this issue Sep 9, 2024 · 1 comment · Fixed by #4824 or #4828
Closed

Update Valgrind version in CI docker image #4759

boquan-fang opened this issue Sep 9, 2024 · 1 comment · Fixed by #4824 or #4828

Comments

@boquan-fang
Copy link
Contributor

boquan-fang commented Sep 9, 2024

Problem:

Older versions of Valgrind can't run with executable compiled by Clang compiler. The current docker image in CI is using Ubuntu 18 with Valgrind version of 3.18.1. That version of Valgrind along with Clang compiled codes will fail the memory check automatically.

Solution:

Requirements / Acceptance Criteria:

  • The up to date docker image should Valgrind 3.20 or higher and can do memory check for Clang compiled code without any other modifications.
@dougch
Copy link
Contributor

dougch commented Sep 12, 2024

Ubuntu22 only ships with valgrind 3.18 so Ubuntu24 it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants