Skip to content

Commit d1844e6

Browse files
christophfroehlichchristophfroehlich
authored
Use global cmake macros and fix gcc-10 build (#1527)
1 parent 996f030 commit d1844e6

File tree

46 files changed

+115
-197
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+115
-197
lines changed

‎ackermann_steering_controller/CMakeLists.txt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
11
cmake_minimum_required(VERSION 3.16)
22
project(ackermann_steering_controller)
33

4-
if(CMAKE_CXX_COMPILER_ID MATCHES "(GNU|Clang)")
5-
add_compile_options(-Wall -Wextra -Wpedantic -Werror=conversion -Werror=unused-but-set-variable
6-
-Werror=return-type -Werror=shadow -Werror=format -Werror=range-loop-construct
7-
-Werror=missing-braces)
8-
endif()
9-
10-
# using this instead of visibility macros
11-
# S1 from https://github.com/ros-controls/ros2_controllers/issues/1053
12-
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
4+
find_package(ros2_control_cmake REQUIRED)
5+
set_compiler_options()
6+
export_windows_symbols()
137

148
# find dependencies
159
set(THIS_PACKAGE_INCLUDE_DEPENDS

‎ackermann_steering_controller/package.xml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@
2323

2424
<build_depend>generate_parameter_library</build_depend>
2525

26+
<build_depend>ros2_control_cmake</build_depend>
27+
2628
<depend>backward_ros</depend>
2729
<depend>control_msgs</depend>
2830
<depend>controller_interface</depend>
@@ -36,7 +38,6 @@
3638
<test_depend>ament_cmake_gmock</test_depend>
3739
<test_depend>controller_manager</test_depend>
3840
<test_depend>hardware_interface_testing</test_depend>
39-
<test_depend>hardware_interface</test_depend>
4041
<test_depend>ros2_control_test_assets</test_depend>
4142

4243
<export>

‎admittance_controller/CMakeLists.txt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
11
cmake_minimum_required(VERSION 3.16)
22
project(admittance_controller)
33

4-
if(CMAKE_CXX_COMPILER_ID MATCHES "(GNU|Clang)")
5-
add_compile_options(-Wall -Wextra -Wpedantic -Werror=conversion -Werror=unused-but-set-variable
6-
-Werror=return-type -Werror=shadow -Werror=format -Werror=range-loop-construct
7-
-Werror=missing-braces)
8-
endif()
9-
10-
# using this instead of visibility macros
11-
# S1 from https://github.com/ros-controls/ros2_controllers/issues/1053
12-
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
4+
find_package(ros2_control_cmake REQUIRED)
5+
set_compiler_options()
6+
export_windows_symbols()
137

148
set(THIS_PACKAGE_INCLUDE_DEPENDS
159
angles

‎admittance_controller/package.xml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@
2222

2323
<buildtool_depend>ament_cmake</buildtool_depend>
2424

25+
<build_depend>ros2_control_cmake</build_depend>
26+
2527
<depend>angles</depend>
2628
<depend>backward_ros</depend>
2729
<depend>control_msgs</depend>

‎bicycle_steering_controller/CMakeLists.txt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
11
cmake_minimum_required(VERSION 3.16)
22
project(bicycle_steering_controller)
33

4-
if(CMAKE_CXX_COMPILER_ID MATCHES "(GNU|Clang)")
5-
add_compile_options(-Wall -Wextra -Wpedantic -Werror=conversion -Werror=unused-but-set-variable
6-
-Werror=return-type -Werror=shadow -Werror=format -Werror=range-loop-construct
7-
-Werror=missing-braces)
8-
endif()
9-
10-
# using this instead of visibility macros
11-
# S1 from https://github.com/ros-controls/ros2_controllers/issues/1053
12-
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
4+
find_package(ros2_control_cmake REQUIRED)
5+
set_compiler_options()
6+
export_windows_symbols()
137

148
# find dependencies
159
set(THIS_PACKAGE_INCLUDE_DEPENDS

‎bicycle_steering_controller/package.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
<buildtool_depend>ament_cmake</buildtool_depend>
2323

2424
<build_depend>generate_parameter_library</build_depend>
25+
<build_depend>ros2_control_cmake</build_depend>
2526

2627
<depend>backward_ros</depend>
2728
<depend>control_msgs</depend>
@@ -35,7 +36,6 @@
3536

3637
<test_depend>ament_cmake_gmock</test_depend>
3738
<test_depend>controller_manager</test_depend>
38-
<test_depend>hardware_interface</test_depend>
3939
<test_depend>hardware_interface_testing</test_depend>
4040
<test_depend>ros2_control_test_assets</test_depend>
4141

‎diff_drive_controller/CMakeLists.txt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
11
cmake_minimum_required(VERSION 3.16)
22
project(diff_drive_controller)
33

4-
if(CMAKE_CXX_COMPILER_ID MATCHES "(GNU|Clang)")
5-
add_compile_options(-Wall -Wextra -Werror=conversion -Werror=unused-but-set-variable
6-
-Werror=return-type -Werror=shadow -Werror=format -Werror=range-loop-construct
7-
-Werror=missing-braces)
8-
endif()
9-
10-
# using this instead of visibility macros
11-
# S1 from https://github.com/ros-controls/ros2_controllers/issues/1053
12-
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
4+
find_package(ros2_control_cmake REQUIRED)
5+
set_compiler_options()
6+
export_windows_symbols()
137

148
set(THIS_PACKAGE_INCLUDE_DEPENDS
159
control_toolbox

‎diff_drive_controller/package.xml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
<author email="[email protected]">Jordan Palacios</author>
2222

2323
<buildtool_depend>ament_cmake</buildtool_depend>
24+
2425
<build_depend>generate_parameter_library</build_depend>
26+
<build_depend>ros2_control_cmake</build_depend>
2527

2628
<depend>backward_ros</depend>
2729
<depend>control_toolbox</depend>

‎effort_controllers/CMakeLists.txt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
11
cmake_minimum_required(VERSION 3.16)
22
project(effort_controllers)
33

4-
if(CMAKE_CXX_COMPILER_ID MATCHES "(GNU|Clang)")
5-
add_compile_options(-Wall -Wextra -Wpedantic -Werror=conversion -Werror=unused-but-set-variable
6-
-Werror=return-type -Werror=shadow -Werror=format -Werror=range-loop-construct
7-
-Werror=missing-braces)
8-
endif()
9-
10-
# using this instead of visibility macros
11-
# S1 from https://github.com/ros-controls/ros2_controllers/issues/1053
12-
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
4+
find_package(ros2_control_cmake REQUIRED)
5+
set_compiler_options()
6+
export_windows_symbols()
137

148
set(THIS_PACKAGE_INCLUDE_DEPENDS
159
forward_command_controller

‎effort_controllers/package.xml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@
1919

2020
<buildtool_depend>ament_cmake</buildtool_depend>
2121

22+
<build_depend>ros2_control_cmake</build_depend>
23+
2224
<depend>backward_ros</depend>
2325
<depend>forward_command_controller</depend>
2426
<depend>pluginlib</depend>

0 commit comments

Comments
 (0)