Skip to content

chore: have uvm_config_image accept a srcmap dict instead of srcs and remap_paths#8796

Merged
basvandijk merged 3 commits intomasterfrom
basvandijk/refactor-uvm_config_image
Feb 12, 2026
Merged

chore: have uvm_config_image accept a srcmap dict instead of srcs and remap_paths#8796
basvandijk merged 3 commits intomasterfrom
basvandijk/refactor-uvm_config_image

Conversation

@basvandijk
Copy link
Collaborator

@basvandijk basvandijk commented Feb 11, 2026

Besides being a requirement for: #8802 it's better to be fully explicit about how source files are mapped into FAT images in the mcopy and uvm_config_image rules. So this replaces their srcs and remap_paths arguments with the single srcmap argument of type label_keyed_string_dict that specifies how a source target is mapped into the FAT image.

@basvandijk basvandijk added the CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 label Feb 11, 2026
@github-actions github-actions bot added the chore label Feb 11, 2026
@basvandijk basvandijk removed the CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 label Feb 11, 2026
@basvandijk basvandijk force-pushed the basvandijk/refactor-uvm_config_image branch from 80177f1 to 852753b Compare February 11, 2026 21:34
@basvandijk basvandijk added the CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 label Feb 11, 2026
@basvandijk basvandijk marked this pull request as ready for review February 11, 2026 22:47
@basvandijk basvandijk requested review from a team as code owners February 11, 2026 22:47
@basvandijk basvandijk added this pull request to the merge queue Feb 12, 2026
Merged via the queue into master with commit cd10260 Feb 12, 2026
39 checks passed
@basvandijk basvandijk deleted the basvandijk/refactor-uvm_config_image branch February 12, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 @consensus @defi @idx @node

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants