manif_INCLUDE_DIRS has duplicate path

[joansola]

The cmake code:

FIND_PACKAGE(manif REQUIRED)
IF(manif_FOUND)
    MESSAGE(STATUS "manif Library FOUND!")
    MESSAGE(STATUS "manif_INCLUDE_DIRS=" ${manif_INCLUDE_DIRS})
ELSEIF(manif_FOUND)
    MESSAGE(STATUS "manif Library NOT FOUND!")
ENDIF(manif_FOUND)

produces

-- manif Library FOUND!
-- manif_INCLUDE_DIRS=/usr/local/include/usr/local/include

where /usr/local/include is appended twice! Then, the line

TARGET_INCLUDE_DIRECTORIES(${PROJECT_NAME} SYSTEM ${manif_INCLUDE_DIRS})

produces not surprisingly

CMake Error at CMakeLists.txt:529 (TARGET_INCLUDE_DIRECTORIES):
  TARGET_INCLUDE_DIRECTORIES called with invalid arguments

[artivis]

This is surprising because I have packages that include manif without any issues. But after checking I do have this duplication too. I'll investigate.

In the meantime, mind that you can also link manif as if it is a library:

target_link_libraries(${PROJECT_NAME} INTERFACE MANIF::manif)

[artivis]

This 'duplication' arise from the fact that manif export both a BUILD_INTERFACE and an INSTALL_INTERFACE in its target_include_directories that is later used to populate manif_INCLUDE_DIRS. Not sure if this is actually an issue/bad practice. It still should work fine tho.