micro-ROS / micro_ros_zephyr_module

micro-ROS Zephyr module and sample code
Apache License 2.0
46 stars 22 forks source link

std_srvs location #106

Closed gseqBE closed 1 year ago

gseqBE commented 1 year ago

Hardware description: nucleo_h743zi RTOS: Installation type: Version or commit hash: Humble

We are moving towards zephyr microros module from the non-module style.

There are a couple of services we are using, specifically trigger style service. There used to be a std_srvs/srv under /mcu_ws/install/include/std_srvs/ in the non-module style and using it as below seemed to work.

image

I'm unable to find the std services in the module solution. Has it moved? This is what my modules/libmicroros/include folder looks like:

image

Thank you!

pablogs9 commented 1 year ago

Here you have the whole list of packages that micro-ROS uses for its building, including msg and srv packages: https://github.com/micro-ROS/micro_ros_zephyr_module/blob/humble/modules/libmicroros/libmicroros.mk#L75-L94

If your required package is not available, please add it manually.

Also, it would be nice to have a way to include "user packages" as in other modules.

gseqBE commented 1 year ago

If I comment out https://github.com/micro-ROS/micro_ros_zephyr_module/blob/humble/modules/libmicroros/libmicroros.mk#L98

or remove the COLCON_IGNORE from: src/common_interfaces/std_srvs folder, so it is not under separate package, but under common interfaces. Is it possible to allow it by default?

pablogs9 commented 1 year ago

Please open a PR with your suggestions.