search

brechtsanders / xlsxio

XLSX I/O - C library for reading and writing .xlsx files
MIT License
397 stars 113 forks source link

[cmake] Export cmake targets #105

Closed JackBoosY closed 2 years ago

JackBoosY commented 2 years ago

Export the targets automaticly according to the cmake configure.

brechtsanders commented 2 years ago

Why do you install xlsxio-config.cmake in share/xlsxio? A more logical location would be lib/cmake/xlsxio, or did you have a specific reason?

JackBoosY commented 2 years ago

Generally, the reason why I used share/xsxio is because vcpkg set it as the cmake configure file's path. I'll change it.

BTW, should we also need to install the cmake find modules such as FindLibZip.cmake since we already used them in CMakeLists.txt?

JackBoosY commented 2 years ago

Ping @brechtsanders for reply.

brechtsanders commented 2 years ago

Published new release with these changes, but I modified it so all .cmake files are in the same folder (currently /cmake under the installation path).