Closed bwrsandman closed 11 months ago
Side note, the version is set in https://github.com/twogood/unshield/blob/550ddfe7344c068f9e77a49710bac4f03166b838/lib/CMakeLists.txt#L33
@bwrsandman Please resolve the conflicts and update your branch, I will have a look then, thanks in advance.
Here are the artifacts that the action generates.
Switched to UPPER
Thank you @bwrsandman and @kratz00 !
convert_utf
as private as it is statically linked so we don't want to export it or its headers.libunshield
include directory in install interface because it is different than in build-timeunshield::
cmake namespace export set namedunshieldConfig
unshieldConfig
export setCMake config sets generate the following files:
${install_dir}/lib/cmake/unshieldunshieldConfig.cmake
${install_dir}/lib/cmake/unshieldunshieldConfig-${build-config}.cmake
An export set allows CMake users to more easily link with a cmake library which unshield now is. They can now in CMake do the following:
Using
add_custom_target(... unshield::unshield...)
can be used as a dependency for another target to generate (extract) filesAlso added artifact upload to the cmake tests.
Resulting tree
ref: https://cmake.org/cmake/help/latest/guide/importing-exporting/index.html#exporting-targets