Skip to content

Idea for CodeChecker toolchain#226

Draft
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:codechecker-toolchain
Draft

Idea for CodeChecker toolchain#226
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:codechecker-toolchain

Conversation

@furtib
Copy link
Copy Markdown
Contributor

@furtib furtib commented May 19, 2026

Why:
We want to use toolchains to determine which CodeChecker binary to use.

What:

  • Created a toolchain for CodeChecker following the Bazel documentation (https://bazel.build/extending/toolchains)
  • Make per_file rule use the toolchain (not the monolithic, since that does not use use_default_shell_env)

Addresses:
none?

Notes:

  • Looking around in other toolchains: rules_go and rules_python it seems toolchains are usually under a folder named the program - in out case CodeChecker (python for rules_python, go for rules_go) - this seems to be to have the toolchain path be //codechecker:toolchain_type instead of //src:toolchain_type.
  • For the local toolchain in the future, I would want to use the path collected in tools.bzl

@furtib furtib requested review from Szelethus and nettle May 19, 2026 09:13
@furtib furtib self-assigned this May 19, 2026
@furtib furtib added the help wanted Extra attention is needed label May 19, 2026
@furtib furtib force-pushed the codechecker-toolchain branch from 17263c7 to d51ce3b Compare May 19, 2026 09:16
@furtib furtib changed the title Add CodeChecker toolchain Idea for CodeChecker toolchain May 19, 2026
@furtib furtib force-pushed the codechecker-toolchain branch from d51ce3b to baaa57a Compare May 19, 2026 09:18
@furtib furtib force-pushed the codechecker-toolchain branch from 1007067 to 2f2f858 Compare May 19, 2026 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant