search

tfussell / xlnt

:bar_chart: Cross-platform user-friendly xlsx library for C++11+
Other
1.49k stars 418 forks source link

Fix XLNT_INCLUDE_INSTALL_DIR #490

Closed StefanRommel closed 4 years ago

StefanRommel commented 4 years ago

Otherwise, XLNT_INCLUDE_INSTALL_DIR is not set correctly and find_package(Xlnt) results to an error:

CMake Error at /usr/lib64/cmake/xlnt/XlntConfig.cmake:23 (message):
  File or directory include referenced by variable XLNT_INCLUDE_DIR does not
  exist !
Call Stack (most recent call first):
  /usr/lib64/cmake/xlnt/XlntConfig.cmake:41 (set_and_check)
  CMakeLists.txt:4 (find_package)

The content of the line 41 of /usr/lib64/cmake/xlnt/XlntConfig.cmake is: set_and_check(XLNT_INCLUDE_DIR "include").

I'm using the Arch Linux user repository package https://aur.archlinux.org/packages/xlnt/. The used build manual is described here: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=xlnt.

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 83.575% when pulling cdb50bbd6b23a29c3d63ef8cc7a267c94920f628 on 50ty:patch-2 into 8d2a8e161b40206d1b4d4b03b1ab4e93ffd4e23d on tfussell:master.

tfussell commented 4 years ago

Thanks for the contribution!