Unverified Commit 85450cc5 authored by Denis Štogl's avatar Denis Štogl Committed by GitHub
Browse files

Merge pull request #18 from ros-controls/clean_cmake

Clean cmake
parent fa923ace
name: Linters
on:
pull_request:
push:
branches:
- master
jobs:
ament_copyright:
......
......@@ -11,10 +11,6 @@ if(NOT CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 14)
endif()
if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
add_compile_options(-Wall -Wextra -Wpedantic)
endif()
# find dependencies
find_package(ament_cmake REQUIRED)
......@@ -22,12 +18,12 @@ install(DIRECTORY include/
DESTINATION include)
if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
ament_lint_auto_find_test_dependencies()
find_package(ament_cmake_gtest REQUIRED)
ament_add_gtest(test_dummy test/test_dummy.cpp)
target_include_directories(test_dummy PRIVATE include)
# ament_target_dependencies(test_dummy rcpputils)
endif()
ament_export_libraries(${PROJECT_NAME})
......
......@@ -13,8 +13,6 @@
<buildtool_depend>ament_cmake</buildtool_depend>
<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>
<export>
<build_type>ament_cmake</build_type>
......
......@@ -11,10 +11,6 @@ if(NOT CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 14)
endif()
if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
add_compile_options(-Wall -Wextra -Wpedantic)
endif()
# find dependencies
find_package(ament_cmake REQUIRED)
......@@ -22,12 +18,12 @@ install(DIRECTORY include/
DESTINATION include)
if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
ament_lint_auto_find_test_dependencies()
find_package(ament_cmake_gtest REQUIRED)
ament_add_gtest(test_dummy test/test_dummy.cpp)
target_include_directories(test_dummy PRIVATE include)
# ament_target_dependencies(test_dummy rcpputils)
endif()
ament_export_libraries(${PROJECT_NAME})
......
......@@ -13,8 +13,6 @@
<buildtool_depend>ament_cmake</buildtool_depend>
<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>
<export>
<build_type>ament_cmake</build_type>
......
......@@ -11,10 +11,6 @@ if(NOT CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 14)
endif()
if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
add_compile_options(-Wall -Wextra -Wpedantic)
endif()
# find dependencies
find_package(ament_cmake REQUIRED)
......@@ -22,12 +18,12 @@ install(DIRECTORY include/
DESTINATION include)
if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
ament_lint_auto_find_test_dependencies()
find_package(ament_cmake_gtest REQUIRED)
ament_add_gtest(test_dummy test/test_dummy.cpp)
target_include_directories(test_dummy PRIVATE include)
# ament_target_dependencies(test_dummy rcpputils)
endif()
ament_export_libraries(${PROJECT_NAME})
......
......@@ -13,8 +13,6 @@
<buildtool_depend>ament_cmake</buildtool_depend>
<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>
<export>
<build_type>ament_cmake</build_type>
......
......@@ -11,10 +11,6 @@ if(NOT CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 14)
endif()
if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
add_compile_options(-Wall -Wextra -Wpedantic)
endif()
# find dependencies
find_package(ament_cmake REQUIRED)
......@@ -22,12 +18,12 @@ install(DIRECTORY config/ description/ launch/
DESTINATION share)
if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
ament_lint_auto_find_test_dependencies()
find_package(ament_cmake_gtest REQUIRED)
ament_add_gtest(test_dummy test/test_dummy.cpp)
target_include_directories(test_dummy PRIVATE include)
# ament_target_dependencies(test_dummy rcpputils)
endif()
ament_package()
......@@ -13,8 +13,6 @@
<buildtool_depend>ament_cmake</buildtool_depend>
<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>
<export>
<build_type>ament_cmake</build_type>
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment