CMake install DIRECTORY include wrong parameters?

[liuzikai]

Hi,

hdlConvertor/src/CMakeLists.txt

Line 96 in 56b2da6

DESTINATION include ${CMAKE_INSTALL_INCLUDEDIR})

According to CMake doc, there should be only one parameter following DESTINATION? When building as a standalone project, ${CMAKE_INSTALL_INCLUDEDIR} is empty so nothing happens. But when I include this project in another one via add_subdirectory with this variable set (also include though), it breaks CMake.

[Nic30]

@liuzikai Do you propose any solution to this problem?
There is a branch where I replaced cmake with meson. https://github.com/Nic30/hdlConvertor/tree/mesonbuild
But I am fighting against the time there...

[liuzikai]

I am not sure either... A possible solution is to remove ${CMAKE_INSTALL_INCLUDEDIR} or default it to "include" if it's empty like the following. Just make sure it doesn't break your python build (which I am not really familiar).

if(CMAKE_INSTALL_INCLUDEDIR STREQUAL "")
    set(CMAKE_INSTALL_INCLUDEDIR "include")
endif()

I was trying to use this library in another pretty large CMake-based project, whose configuration does some wired things to these variables. Simply remove either include or ${CMAKE_INSTALL_INCLUDEDIR} works, although later I encountered some other problems and not using this lib for now. So anyway, feel free to apply these easy fixes or just close this issue as you like ;)