micro-ROS / micro_ros_zephyr_module

micro-ROS Zephyr module and sample code
Apache License 2.0
54 stars 26 forks source link

Fix/zephyr includes (backport #82) #85

Closed mergify[bot] closed 2 years ago

mergify[bot] commented 2 years ago

This is an automatic backport of pull request #82 done by Mergify.


Mergify commands and options
More conditions and actions can be found in the [documentation](https://docs.mergify.com/). You can also trigger Mergify actions by commenting on this pull request: - `@Mergifyio refresh` will re-evaluate the rules - `@Mergifyio rebase` will rebase this PR on its base branch - `@Mergifyio update` will merge the base branch into this PR - `@Mergifyio backport ` will backport this PR on `` branch Additionally, on Mergify [dashboard](https://dashboard.mergify.com/) you can: - look at your merge queues - generate the Mergify configuration with the config editor. Finally, you can contact us on https://mergify.com
pablogs9 commented 2 years ago

Hello @uLipe, this is failing here: https://github.com/ros2/rcutils/blob/2d0c9e2e94080c4a207057cd4f1b5804766a1633/src/time_unix.c#L30

It is using the latest main rcutils and it has this PR: https://github.com/ros2/rcutils/pull/383 Any idea?

uLipe commented 2 years ago

@pablogs9 yes, the KERNELVERSION macro is not used in zephyr 2.7.2 I replaced it with ZEPHYR_VERSION_CODE which is compatible across both releases.

Fix already present: https://github.com/ros2/rcutils/pull/390

pablogs9 commented 2 years ago

@uLipe last rcutils fixs the issue. Thanks a lot!