-
Notifications
You must be signed in to change notification settings - Fork 100
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
Apt install version has old state #259
Comments
Worry not. Releases don't make themselves.
When in doubt whether some change should be expected to be in the latest
apt release or not, check the repo history for a version number. The change
you are looking for hasn't been released yet as it is above the last
version tag in the history.
https://github.com/ros-controls/control_toolbox/commits/ros2-master/
…On Thu, 2 Jan 2025, 14:37 KevVincent, ***@***.***> wrote:
Hey I tried installing the repo using apt and i am getting the error while
building my package that depends on control_toolbox
opt/ros/jazzy/include/realtime_tools/realtime_tools/realtime_buffer.h:25:2:
error: #warning "This header include is deprecated. Please update your code
to use 'realtime_buffer.hpp' header." [-Werror=cpp] 25 | #warning \ |
^~~~~~~ In file included from![Screenshot from 2025-01-02 15-36-13](
https://github.com/user-attachments/assets/a84df141-4b3e-4b41-970a-84c376d42be8):46:
/opt/ros/jazzy/include/realtime_tools/realtime_tools/realtime_publisher.h:25:2:
error: #warning "This header include is deprecated. Please update your code
to use 'realtime_publisher.hpp' header." [-Werror=cpp] 25 | #warning \ |
^~~~~~~
if i include
/opt/ros/jazzy/include/control_toolbox/control_toolbox/pid_ros.hpp:45.
I see that the change was made two weeks ago #247
<#247> but the apt
install version's pid_control.hpp
Screenshot.from.2025-01-02.15-36-13.png (view on web)
<https://github.com/user-attachments/assets/e37550b6-6081-4f76-a644-b90ab1620ba0>
Any help would be appreciated. Thanks in advance.
BR,
Kevin
—
Reply to this email directly, view it on GitHub
<#259>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA24PYITGM4TB7MA37WA6H32IVFLLAVCNFSM6AAAAABUQCHII2VHI2DSMVQWIX3LMV43ASLTON2WKOZSG43DMMJTHE3TQNA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
This means you can either
|
22 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey I tried installing the repo using apt and i am getting the error while building my package that depends on control_toolbox
opt/ros/jazzy/include/realtime_tools/realtime_tools/realtime_buffer.h:25:2: error: #warning "This header include is deprecated. Please update your code to use 'realtime_buffer.hpp' header." [-Werror=cpp] 25 | #warning \ | ^~~~~~~ In file included from![Screenshot from 2025-01-02 15-36-13](https://github.com/user-attachments/assets/a84df141-4b3e-4b41-970a-84c376d42be8):46: /opt/ros/jazzy/include/realtime_tools/realtime_tools/realtime_publisher.h:25:2: error: #warning "This header include is deprecated. Please update your code to use 'realtime_publisher.hpp' header." [-Werror=cpp] 25 | #warning \ | ^~~~~~~
if i include /opt/ros/jazzy/include/control_toolbox/control_toolbox/pid_ros.hpp:45.
I see that the change was made two weeks ago #247 but the apt install version's pid_control.hpp
Any help would be appreciated. Thanks in advance.
BR,
Kevin
The text was updated successfully, but these errors were encountered: