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.

Sign up for GitHub

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

Jump to bottom

[WOP] #6060

Closed
wants to merge 5 commits into from
Closed

[WOP] #6060

wants to merge 5 commits into from

Commits on Mar 20, 2024

  1. common: cleanup the way Valgrind is disabled in build 

    Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
    @grom72
    grom72 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    e3f04b0 View commit details
    Browse the repository at this point in the history

  2. common: disable Valgrind tools when Valgrind is not enabled 

    Disable all Valgrind tools when Valgrind is not enabled
Use VALGRIND=0 env. var. to disable Valgrind in compilation

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
    @grom72
    grom72 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    fb28242 View commit details
    Browse the repository at this point in the history

  3. common: separate test to verify if Valgrind enabled in tests 

    Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
    @grom72
    grom72 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    e53bbb3 View commit details
    Browse the repository at this point in the history

  4. common: check if Valgrind was enabled during biuld time 

    Test is skipped if any Valgrind tool is required and
Valgrind was not enabled during build (VALGRIND=0)

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
    @grom72
    grom72 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    ecfaecb View commit details
    Browse the repository at this point in the history

  5. common: add GHA test w/o Valgrind 

    disable valgrind in builds that do not need them

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
    @grom72
    grom72 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    6338a7f View commit details
    Browse the repository at this point in the history