search

martinmoene / span-lite

span lite - A C++20-like span for C++98, C++11 and later in a single-file header-only library
Boost Software License 1.0
497 stars 41 forks source link

Improve Cmake/Conan packaging #37

Closed pleroux0 closed 5 years ago

pleroux0 commented 5 years ago

Some changes to improve CMake/Conan packages. Related to points 3 and 4 of #4.

  1. Currently, the CMake install installs the package as either a 32bit or 64bit package. This is a header-only library so this isn't necessary.
  2. It installs the CMake targets to cmake/span-lite/span-lite-*.cmake. The previous location is not searched when running find_package from CMake.
  3. Run CMake install instead of copying files. This allows Conan/CMake projects to use find_package(span-lite) just like pure CMake projects. Makes where the dependency is coming from transparent.

Let me know if you want any adjustments or require any changes.

pleroux0 commented 5 years ago

The failure from AppVeyor is from their side and is unrelated to my changes. The CI just needs to be started again.

martinmoene commented 5 years ago

Thanks @pleroux0 . Looks good to me (inso(not very)far as I'm able to judge ;)