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

[bazel] Add QEMU execution environment #25909

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

Conversation

jwnrt
Copy link
Contributor

@jwnrt jwnrt commented Jan 16, 2025

This is an early draft subject to lots of changes.

jwnrt and others added 3 commits January 16, 2025 13:12
This version allows us to select a different python toolchain (e.g.
"python3.12") for different `py_binary`s.

The `rules_cc` is transitively updated, so I've fixed the version in
`MODULE.bazel` to match.

Signed-off-by: James Wainwright <[email protected]>
Needed for QEMU scripts which use newer features. We don't want to bump
our default Python version because hardware engineers need to run those
scripts outside of Bazel and 3.12 isn't available on all platforms that
we support. We would have to start requiring `venv`s for hardware
development.

Requires adding `python_version: 3.9` to the existing `py_binary`s
otherwise any transition to the 3.12 target will carry over when then
evaluating these 3.9-specific binaries and cause them to not find their
dependencies correctly.

Signed-off-by: James Wainwright <[email protected]>
Signed-off-by: James Wainwright <[email protected]>
Co-authored-by: Amaury Pouly <[email protected]>
@jwnrt jwnrt force-pushed the qemu branch 5 times, most recently from 5d90b3b to da4af36 Compare January 16, 2025 15:34
jwnrt and others added 3 commits January 16, 2025 15:56
Signed-off-by: James Wainwright <[email protected]>
Co-authored-by: Amaury Pouly <[email protected]>
Signed-off-by: James Wainwright <[email protected]>
Co-authored-by: Amaury Pouly <[email protected]>
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.

1 participant