Skip to content

Commit

Permalink
Use linters when testing.
Browse files Browse the repository at this point in the history
ament_lint_auto and ament_lint_common are both already dependencies
listed in this package but they were not yet enabled.

With these linters enabled there are a number of failures.
  • Loading branch information
nuclearsandwich committed May 6, 2019
1 parent 4a9d908 commit 464fa4f
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions rcl_logging_log4cxx/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,11 @@ ament_target_dependencies(${PROJECT_NAME}

target_compile_definitions(${PROJECT_NAME} PRIVATE "RCL_LOGGING_BUILDING_DLL")

if (BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
ament_lint_auto_find_test_dependencies()
endif()

install(TARGETS ${PROJECT_NAME}
ARCHIVE DESTINATION lib
LIBRARY DESTINATION lib
Expand Down

0 comments on commit 464fa4f

Please sign in to comment.