-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KPE-330-pendulum executor ROS2 refactorization #2
base: galactic
Are you sure you want to change the base?
Conversation
c5dbc4c
to
f986def
Compare
c7064a8
to
f4f3246
Compare
f4f3246
to
520f44a
Compare
menu for pendulum demo to choose between the different threaded executors kpsr executor in pendulum demo included
e5bd19c
to
dd431cb
Compare
find_package(kpsr_ros2_executor REQUIRED) | ||
find_package(Klepsydra REQUIRED) | ||
find_package(KlepsydraAdmin REQUIRED) | ||
find_package(KlepsydraStreaming REQUIRED) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
provide the versions to be sure that you are using the right API. admin and streaming have changed a lot recently, so version numbers are critical here
find_package(KlepsydraAdmin REQUIRED) | ||
find_package(KlepsydraStreaming REQUIRED) | ||
|
||
set(CMAKE_THREAD_LIBS_INIT "-lpthread") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the better way to do this is find_package(Threads)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Squash all commits, update fork and then merge
No description provided.