Conversation
Enables addition of new colcon built packages in this repo
|
Depends on #74 |
|
These changes split from the original PR #71 and has all changes requested for the cmake macro from that review |
|
jacobperron
left a comment
There was a problem hiding this comment.
Thanks for splitting out the cmake package 👍
| endif() | ||
| find_program(PROGRAM ros2) | ||
| if (NOT PROGRAM) | ||
| message("Unable to find ros2cli, have you sourced your ros setup files?") |
There was a problem hiding this comment.
We should add ros2cli as a build and build_export dependency to the package.xml so we don't have to rely on sourcing a setup file in the case that we're building alongside ros2cli.
|
@ross-desmond Just as a housekeeping note, we're moving forward on making a release for the patch, and this will get in the first sync after the patch release. |
Additionally update build dependencies.
|
Awesome, making changes... |
|
Disregard that CI, I think I used the wrong repos file... |
Looks like all changes are accounted for. |
|
CI up to
|
Adds cmake macros for generating keys and keystore for nodes at compile time.