Skip to content

Demo pip codechecker - #293

Draft
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:codechecker-pip
Draft

Demo pip codechecker#293
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:codechecker-pip

Conversation

@furtib

@furtib furtib commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

This PR is a bit premature.

Why:
We want a hermetic toolchain.

What:

  • Added a way to obtain codechecker through pip.
  • For some reason, we need to set the CC_BIN_DIR environment variable to point to the codechecker's config directory. (This part needs some work... as you can see on CI)

Notes:
Depends on: #274

Addresses:
none?

@furtib furtib self-assigned this Aug 17, 2026
@furtib furtib added the invalid This doesn't seem right label Aug 17, 2026

@nettle nettle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking about something like this!

Comment thread requirements_lock.txt
@@ -0,0 +1,47 @@
alembic==1.19.1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OMG, is all of this hm.. stuff really needed for CodeChecker?

Comment thread MODULE.bazel
Comment on lines +49 to +55
pip = use_extension("@rules_python//python/extensions:pip.bzl", "pip")
pip.parse(
hub_name = "pip",
python_version = "3.12",
requirements_lock = "//:requirements_lock.txt",
)
use_repo(pip, "pip")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is how it should be done

@furtib

furtib commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

Okay, now it works better in CI. (I have no idea why the length of running tests in CI doubled)
I needed to add a new element to the toolchain: codechecker_files_to_run.
I cannot integrate this into the run_files depset because its content is a different type, even though its a very similar concept.

@furtib furtib removed the invalid This doesn't seem right label Aug 27, 2026
@furtib
furtib requested review from Szelethus and nettle August 27, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants