Skip to content

Commit 9646a40

Browse files
committed
Fix ament_lint_auto settings for cppcheck and uc
1 parent d3fb3bd commit 9646a40

File tree

2 files changed

+12
-20
lines changed

2 files changed

+12
-20
lines changed

tf2_geometry_msgs/CMakeLists.txt

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -29,19 +29,14 @@ ament_auto_find_build_dependencies(REQUIRED ${required_dependencies})
2929

3030
if(BUILD_TESTING)
3131
find_package(ament_cmake_gtest REQUIRED)
32+
find_package(ament_lint_auto REQUIRED)
3233
find_package(rclcpp REQUIRED)
33-
34-
# TODO(ros2/geometry2#259) Switch to ament_lint_auto once headers
34+
35+
# TODO(ros2/geometry2#259) Remove once headers
3536
# are renamed to .hpp
36-
find_package(ament_cmake_cppcheck REQUIRED)
37-
find_package(ament_cmake_cpplint REQUIRED)
38-
find_package(ament_cmake_lint_cmake REQUIRED)
39-
find_package(ament_cmake_uncrustify REQUIRED)
40-
41-
ament_cppcheck(LANGUAGE "c++")
42-
ament_cpplint()
43-
ament_lint_cmake()
44-
ament_uncrustify(LANGUAGE "c++")
37+
set(ament_cmake_uncrustify_ADDITIONAL_ARGS -l CPP)
38+
set(ament_cmake_cppcheck_LANGUAGE c++)
39+
ament_lint_auto_find_test_dependencies()
4540

4641
ament_add_gtest(test_tf2_geometry_msgs test/test_tf2_geometry_msgs.cpp)
4742
if(TARGET test_tf2_geometry_msgs)

tf2_kdl/CMakeLists.txt

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,18 +31,15 @@ install(DIRECTORY include/${PROJECT_NAME}/
3131

3232
if(BUILD_TESTING)
3333
find_package(ament_cmake_gtest REQUIRED)
34+
find_package(ament_lint_auto REQUIRED)
3435
find_package(rclcpp REQUIRED)
3536
find_package(tf2_msgs REQUIRED)
36-
37-
# TODO(ros2/geometry2#259) Switch to ament_lint_auto once headers
37+
38+
# TODO(ros2/geometry2#259) Remove once headers
3839
# are renamed to .hpp
39-
find_package(ament_cmake_cppcheck REQUIRED)
40-
find_package(ament_cmake_cpplint REQUIRED)
41-
find_package(ament_cmake_lint_cmake REQUIRED)
42-
find_package(ament_cmake_uncrustify REQUIRED)
43-
44-
ament_cppcheck(LANGUAGE "c++")
45-
ament_uncrustify(LANGUAGE "c++")
40+
set(ament_cmake_uncrustify_ADDITIONAL_ARGS -l CPP)
41+
set(ament_cmake_cppcheck_LANGUAGE c++)
42+
ament_lint_auto_find_test_dependencies()
4643

4744
ament_add_gtest(test_kdl test/test_tf2_kdl.cpp)
4845
target_include_directories(test_kdl PUBLIC

0 commit comments

Comments
 (0)