Skip to content

Patch rosidl to add missing cstdint include#350

Merged
traversaro merged 3 commits intoRoboStack:mainfrom
noahjutz:patch-rosidl-cstdint
Sep 18, 2025
Merged

Patch rosidl to add missing cstdint include#350
traversaro merged 3 commits intoRoboStack:mainfrom
noahjutz:patch-rosidl-cstdint

Conversation

@noahjutz
Copy link
Contributor

Closes #349

@Tobias-Fischer
Copy link
Contributor

You also need to bump the version number as for example in https://github.com/noahjutz/ros-humble/blob/6eb47fb8d11aa7da8873147d84c86970639e9648/pkg_additional_info.yaml#L148

@traversaro
Copy link
Member

traversaro commented Sep 18, 2025

@noahjutz we should document definitely this better, but basically all robostack packages share the same build number, that is set globally in https://github.com/RoboStack/ros-humble/blob/main/vinca.yaml#L9 (and is raise at full rebuild/sync). So currently the latest rosidl package has build number 13, to ensure that a newly built package is installed instead of the old one, it needs to have build number 14 .

TL;DR: Use 14 instead of 1

@traversaro
Copy link
Member

Thanks!

@traversaro traversaro merged commit ad99f44 into RoboStack:main Sep 18, 2025
6 checks passed
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.

rosidl: missing <cstdint> include causes compile failure

3 participants