Repository navigation
Per-package files and dependency fixes next to the package's other settings - #224
Open
Tobias-Fischer wants to merge 1 commit into
Open
Tobias-Fischer wants to merge 1 commit into
Tobias-Fischer wants to merge 1 commit into
Conversation
This was referenced Nov 14, 2026
…ttings - Dependency fixes live in pkg_additional_info.yaml: each layer's patch/dependencies.yaml is merged into it, so a package's add_*/remove_* keys sit next to its build_number and cmake args. Both files were keyed, layered and merged per package the same way; the merged files hold exactly what the two did (compared per package and key). import_distro merges an old repository's two files; the sort task, docs and skills follow. - ros_workspace's activation scripts live here (shared/files/ros2-ros-workspace/): vinca-rs copies a layer's files/<pkg>/ into that package's recipe. - check reports entries for packages the distribution (or, for shared/, no distribution) doesn't have: typos and leftovers. It found three in rolling, now removed (grid_map_octomap, gz_launch_vendor, slam_toolbox). A vinca-rs solve failure that isn't the solver's answer raises instead of counting as a conflict. - vinca-rs 0.1.9, which reads both (Tobias-Fischer/vinca-rs#7, #8) and builds ament_cmake with Ninja on Windows. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Tobias-Fischer
force-pushed
the
vinca-rs/4-package-files
branch
from
November 14, 2026 12:07
7b61d9f to
c436914
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A package's extra files and dependency fixes sit next to its other settings.
pkg_additional_info.yaml: each layer'spatch/dependencies.yamlis merged into it, so a package'sadd_*/remove_*keys sit next to itsbuild_numberand CMake args. Both files were keyed, layered and merged per package the same way; the merged files hold exactly what the two did (compared per package and key: the merge alone leaves the generated recipes unchanged; the vinca-rs bump below changes the Windows build scripts).import_distromerges an old repository's two files; the sort task, docs and skills follow.shared/files/ros2-ros-workspace/): vinca-rs copies a layer'sfiles/<pkg>/into that package's recipe.rs checkreports entries for packages the distribution (or, forshared/, no distribution) doesn't have: typos and leftovers. It found three in rolling, now removed (grid_map_octomap, gz_launch_vendor, slam_toolbox). A vinca-rs solve failure that isn't the solver's answer raises instead of counting as a conflict.Merging with #227: #227 edits
shared/patch/dependencies.yaml, which this PR merges intopkg_additional_info.yaml. Whichever of the two lands second puts #227'sremove_host: ["wxpython"]into the existingmrpt_guientry ofshared/pkg_additional_info.yaml, and its build numbers into the existingmrpt_guientries of the distributions'pkg_additional_info.yaml(jazzy merges without a conflict but would end up with twomrpt_gui:keys).A follow-up: #220 (one per-package folder for patches, tests and files).
🤖 Generated with Claude Code