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

Pin the number of jobs in some Make targets #3640

Merged
merged 1 commit into from
Mar 31, 2022

Commits on Mar 31, 2022

  1. build-logger tests should not depend on the 'all' target

    If someone runs `make -j16 test_unit test_analyzer test_web_sqlite`,
    then the command would spuriously fail due to a filesystem
    race-condition.
    
    The `cp ldlogger build/bin` would basically try to copy a
    not-yet-existing file,
    causing a failure immediately, or later in the test pipeline, when it
    exercises the logger. Causing an error similar to this:
    ```
    make[1]: Leaving directory `codechecker/analyzer'
    make: *** [test_analyzer] Error 2
    make: *** Waiting for unfinished jobs....
    ```
    
    Let's work around the issue by omitting the `all` target from the
    `test*` targets. This way if it's already built, tests should just pass.
    Balazs Benics committed Mar 31, 2022
    Configuration menu
    Copy the full SHA
    cd88ccd View commit details
    Browse the repository at this point in the history