Skip to content

Demo pip codechecker - #293

Draft
furtib wants to merge 1 commit into
Ericsson:mainfrom
furtib:codechecker-pip
Draft

Demo pip codechecker#293
furtib wants to merge 1 commit 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: Add dependencies of tools from toolchain to runfiles #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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants