Skip to content
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

Declare dependency of gazebo_hardware_plugins to urdf in CMakeLists.txt #117

Merged

Conversation

hobbeshunter
Copy link
Contributor

Since gazebo_system_interface.hpp includes urdf/model:

https://github.com/ros-simulation/gazebo_ros2_control/blob/698d01381423cea3232a78ab0ada7eb12cd6d40b/gazebo_ros2_control/include/gazebo_ros2_control/gazebo_system_interface.hpp#L32

I guess also the CMake target gazebo_hardware_plugins should depend on urdf.

At least that fixed my build on Ubuntu 18.04.

gazebo_ros2_control/CMakeLists.txt Outdated Show resolved Hide resolved
@hobbeshunter
Copy link
Contributor Author

Like this?

Copy link
Collaborator

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are two things more:

  • remove the same header from this file gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp
  • Remove the urdf depend in the package.xml

@hobbeshunter
Copy link
Contributor Author

Ah sry. Misread the .cpp file ending as .hpp.

@ahcorde ahcorde merged commit c89584d into ros-controls:galactic Mar 17, 2022
@ahcorde
Copy link
Collaborator

ahcorde commented Mar 17, 2022

Thank you for your contribution @hobbeshunter

@hobbeshunter
Copy link
Contributor Author

You're welcome.

I probably should stop coding during night time 😆

@hobbeshunter hobbeshunter deleted the fix/missing-urdf-dependency branch March 17, 2022 19:58
ahcorde added a commit that referenced this pull request Apr 22, 2022
…xt (#117) (#119)

Co-authored-by: Martin Wudenka <Martin.Wudenka@gmx.de>
ksotebeer pushed a commit to ksotebeer/gazebo_ros2_control that referenced this pull request Jun 14, 2022
ahcorde added a commit that referenced this pull request Nov 21, 2022
* fix maintainer email (#92)

Signed-off-by: ahcorde <ahcorde@gmail.com>

* CHANGELOG

Signed-off-by: ahcorde <ahcorde@gmail.com>

* 0.0.4

* 0.0.5

* Update Foxy dockerfile (#101)

Signed-off-by: ahcorde <ahcorde@gmail.com>

* Declare dependency of gazebo_hardware_plugins to urdf in CMakeLists.txt (#117) (#119)

Co-authored-by: Martin Wudenka <Martin.Wudenka@gmx.de>

* Added diff drive example (#113)

Signed-off-by: ahcorde <ahcorde@gmail.com>

* changelog

Signed-off-by: ahcorde <ahcorde@gmail.com>

* 0.1.0

* [backport Foxy] #136 #147 #139 #134 (#150)

* Implemented perform_command_mode_switch override in GazeboSystem (#136)
* added namespace to controller manager (#147)
* Added logic for activating hardware interfaces (#139)

Signed-off-by: ahcorde <ahcorde@gmail.com>
Co-authored-by: Keegan Sotebeer <ksotebeer95@gmail.com>
Co-authored-by: Maciej Bednarczyk <83034299+mcbed@users.noreply.github.com>
Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>

* [Backport Foxy] Support for mimic joints and example with gripper. (#121)

* [Backport Foxy] Support for mimic joints and example with gripper.

* Enable loading params from multiple yaml files (#149) (#151)

Co-authored-by: Tony Najjar <tony.najjar@logivations.com>

* Adding simulation time parameter for the controller manager (#138) (#158)

Co-authored-by: Eslam Salah <eslamsalahelsheikh@gmail.com>

* Changelog

Signed-off-by: ahcorde <ahcorde@gmail.com>

* 0.1.1

* Add pre-commit hooks configuration (#161)

* Add pre-commit hooks configuration

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Pre-commit in a separate action

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix codespell

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Martin Wudenka <Martin.Wudenka@gmx.de>
Co-authored-by: Keegan Sotebeer <ksotebeer95@gmail.com>
Co-authored-by: Maciej Bednarczyk <83034299+mcbed@users.noreply.github.com>
Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
Co-authored-by: Leander Stephen D'Souza <leanderdsouza1234@gmail.com>
Co-authored-by: Tony Najjar <tony.najjar@logivations.com>
Co-authored-by: Eslam Salah <eslamsalahelsheikh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants