Skip to content

Commit

Permalink
Fix include paths order
Browse files Browse the repository at this point in the history
We used to include `.` and `include/`, but the former is no longer
necessary and the latter has to have the highest priority to avoid that
installed headers take higher precedence, leading to a build employing
outdated headers.
  • Loading branch information
aleclearmind committed Dec 13, 2018
1 parent 9d13ce3 commit a292542
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ cmake_minimum_required(VERSION 2.8)

project(revamb)

# This has to be first to get highest priority
include_directories(include/)

# Doxygen
find_package(Doxygen)
if(DOXYGEN_FOUND)
Expand Down Expand Up @@ -136,8 +139,6 @@ endif()

set(CMAKE_INSTALL_RPATH "\$ORIGIN/../lib${LLVM_LIBDIR_SUFFIX}")

include_directories(./ include/)

add_subdirectory(lib)
add_subdirectory(tools)

Expand Down

0 comments on commit a292542

Please sign in to comment.