Closed luncliff closed 2 years ago
Hi, I'm working on https://github.com/microsoft/vcpkg/pull/20403 and the maintainer suggested to apply this changes to upstream. Would you review this?
New build option, BUILD_TESTING can enable/disable test executables. The option is ON by default, so normal build will remain same.
BUILD_TESTING
ON
install
find_package
Now the CMakeLists uses install. After the install, users can search gemmlowp headers and libraries with find_package
find_package(gemmlowp CONFIG) target_link_libraries(main PRIVATE gemmlowp::gemmlowp)
Please check the updated .travis.yml file.
.travis.yml
https://github.com/microsoft/vcpkg/pull/20403
Hi, I'm working on https://github.com/microsoft/vcpkg/pull/20403 and the maintainer suggested to apply this changes to upstream. Would you review this?
Changes
Make CTest targets optional
New build option,
BUILD_TESTINGcan enable/disable test executables. The option isONby default, so normal build will remain same.Support CMake
installandfind_packageNow the CMakeLists uses
install. After the install, users can search gemmlowp headers and libraries withfind_packageTravis CI build job to run CTest targets
Please check the updated
.travis.ymlfile.Related Issues
https://github.com/microsoft/vcpkg/pull/20403
Previous Works
63