build: exclude libmultiprocess test target from compile commands #35401

pull Sanjana2906 wants to merge 1 commits into bitcoin:master from Sanjana2906:fix/iwyu-libmultiprocess changing 1 files +5 −0
  1. Sanjana2906 commented at 8:10 AM on May 28, 2026: none

    Fixes #35361

    The mptest target includes generated .capnp.h files that don't exist at CMake configure time. When IWYU reads compile_commands.json, it sees these non-existent files and fails with errors like: fatal error: 'mp/test/foo.capnp.h' file not found This PR excludes the mptest target from the compilation database by setting EXPORT_COMPILE_COMMANDS OFF, following the same pattern already applied to the libmultiprocess example targets in PR #31741.

    Before this change:

    • IWYU would fail on libmultiprocess test files
    • CI logs show sporadic "file not found" errors

    After this change:

    • The test target is excluded from compile_commands.json
    • IWYU no longer attempts to check these files
    • Cleaner CI logs with no false positive errors
  2. build: exclude libmultiprocess test target from compile commands 658a054ff0
  3. DrahtBot added the label Build system on May 28, 2026
  4. DrahtBot commented at 8:10 AM on May 28, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35401.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. DrahtBot added the label CI failed on May 28, 2026
  6. DrahtBot commented at 8:20 AM on May 28, 2026: contributor

    <!--85328a0da195eb286784d51f73fa0af9-->

    🚧 At least one of the CI tasks failed. <sub>Task lint: https://github.com/bitcoin/bitcoin/actions/runs/26562892028/job/78250708763</sub> <sub>LLM reason (✨ experimental): CI failed the lint “subtree” check because a subtree directory was modified without a corresponding subtree merge.</sub>

    <details><summary>Hints</summary>

    Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

    </details>

  7. DrahtBot commented at 8:20 AM on May 28, 2026: contributor

    Thx, but according to the CI failure, this is the wrong repo. The subtree will have to be fixed upstream.

  8. DrahtBot closed this on May 28, 2026


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-31 17:50 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me