Skip to content

Commit 378dfb2

Browse files
committed
fix: move dynmsg to unix builds and addapt the patch to make that work.
1 parent cd7e104 commit 378dfb2

2 files changed

Lines changed: 14 additions & 1 deletion

File tree

patch/ros-humble-dynmsg.patch

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,15 @@ index 072819c..ed4716f 100644
181181
case rosidl_typesupport_introspection_cpp::ROS_TYPE_MESSAGE:
182182
// For nested types, don't copy the data out of the buffer directly. Recursively read the
183183
// nested type into the YAML.
184+
@@ -398,7 +398,7 @@ dynamic_array_to_yaml_impl_bool(
185+
static_cast<void>(member_info);
186+
for (size_t ii = 0; ii < v->size(); ++ii) {
187+
DYNMSG_DEBUG(std::cout << (v->operator[](ii) ? "true" : "false") << ", ");
188+
- array_node.push_back(v->operator[](ii));
189+
+ array_node.push_back(static_cast<bool>(v->operator[](ii)));
190+
}
191+
DYNMSG_DEBUG(std::cout << std::endl);
192+
}
184193
@@ -509,12 +509,12 @@ dynamic_array_to_yaml(
185194
reinterpret_cast<const std::vector<std::string> *>(member_data),
186195
array_node);

vinca.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ packages_select_by_deps:
139139
# These are emscripten-wasm32 specific packages
140140
- if: wasm32
141141
then:
142-
- dynmsg
143142
- rmw_wasm_cpp
144143
- test_wasm
145144
- wasm_cpp
@@ -353,6 +352,11 @@ packages_select_by_deps:
353352

354353
- pointcloud_to_laserscan
355354

355+
# These packages specifically don't work on windows
356+
- if: not win
357+
then:
358+
- dynmsg
359+
356360
# These packages are only built on Linux as they depend on Linux-specific API
357361
- if: linux
358362
then:

0 commit comments

Comments
 (0)