Skip to content

build(vendor): add test-only gating metadata for vendored messages#625

Draft
esteve wants to merge 2 commits intoros2-rust:mainfrom
esteve:vendored-messages-only-for-tests
Draft

build(vendor): add test-only gating metadata for vendored messages#625
esteve wants to merge 2 commits intoros2-rust:mainfrom
esteve:vendored-messages-only-for-tests

Conversation

@esteve
Copy link
Copy Markdown
Collaborator

@esteve esteve commented Apr 13, 2026

This PR avoids including the test_msgs and example_interfaces packages when not building rclrs for testing.

@esteve
Copy link
Copy Markdown
Collaborator Author

esteve commented Apr 13, 2026

This should fix #550 and maybe #616

@esteve esteve requested review from maspe36 and mxgrey April 13, 2026 21:07
@esteve esteve force-pushed the vendored-messages-only-for-tests branch 2 times, most recently from 666bf77 to 7ec5352 Compare April 13, 2026 21:09
@esteve esteve marked this pull request as draft April 13, 2026 21:35
@esteve esteve removed request for maspe36 and mxgrey April 13, 2026 21:35
@esteve
Copy link
Copy Markdown
Collaborator Author

esteve commented Apr 13, 2026

Back to draft to figure out a better solution.

@esteve esteve force-pushed the vendored-messages-only-for-tests branch 3 times, most recently from e7d83cc to a8070ba Compare April 15, 2026 09:32
Signed-off-by: Esteve Fernandez <[email protected]>
@esteve esteve force-pushed the vendored-messages-only-for-tests branch 2 times, most recently from 59f43d6 to d58d085 Compare April 15, 2026 15:52
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