Skip to content

Commit

Permalink
move find_package or test dependencies to test block (#331)
Browse files Browse the repository at this point in the history
  • Loading branch information
wjwwood authored Jun 28, 2018
1 parent b0f7c11 commit 5e65083
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions rviz_rendering/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
endif()

find_package(ament_cmake REQUIRED)
find_package(ament_cmake_gtest REQUIRED)
find_package(ament_cmake_gmock REQUIRED)

# Qt5 boilerplate options from http://doc.qt.io/qt-5/cmake-manual.html
set(CMAKE_INCLUDE_CURRENT_DIR ON)
Expand Down Expand Up @@ -177,6 +175,9 @@ install(DIRECTORY "${CMAKE_SOURCE_DIR}/ogre_media"
USE_SOURCE_PERMISSIONS)

if(BUILD_TESTING)
find_package(ament_cmake_gmock REQUIRED)
find_package(ament_cmake_gtest REQUIRED)

# TODO(wjwwood): replace this with ament_lint_auto() and/or add the copyright linter back
find_package(ament_cmake_cppcheck REQUIRED)
find_package(ament_cmake_cpplint REQUIRED)
Expand Down

0 comments on commit 5e65083

Please sign in to comment.