feat(lib): CMake install/export for the host C++ library (find_package + FetchContent/CPM)

[finger563]

What & why

The host C++ library under lib/ had no consumable install. This adds a proper modern-CMake install + export so an external project can use it two ways with the same link name espp::espp:

• find_package(espp REQUIRED) from an install tree, and
• add_subdirectory(lib) via FetchContent / CPM from the build tree.

Changes

• lib/CMakeLists.txt: espp::espp ALIAS (+ EXPORT_NAME espp), $<BUILD_INTERFACE>/$<INSTALL_INTERFACE> include dirs, PUBLIC cxx_std_23 + system libs. Gated all pybind11 discovery + the Python module behind a new option(ESPP_BUILD_PYTHON OFF) (if(SKBUILD OR ESPP_BUILD_PYTHON)), so a plain cmake -S lib builds only the C++ lib + export with no pybind11/network dependency.
• lib/espp.cmake: espp_install_cmake_package() — install(TARGETS ... EXPORT esppTargets), installs public headers, install(EXPORT ... NAMESPACE espp::), generates + installs esppConfig.cmake + esppConfigVersion.cmake (SameMajorVersion).
• lib/cmake/esppConfig.cmake.in: @PACKAGE_INIT@, find_dependency(Threads), include targets, check_required_components.
• lib/build.sh / build.ps1: pass -DESPP_BUILD_PYTHON=ON to preserve their (and CI's) Python-producing behavior; the scikit-build wheel path (SKBUILD) is unaffected.

Consumer usage

find_package:

find_package(espp REQUIRED)
target_link_libraries(consumer PRIVATE espp::espp)

FetchContent / CPM (note SOURCE_SUBDIR lib, and a recursive submodule checkout is required since third-party deps are vendored):

CPMAddPackage(NAME espp GITHUB_REPOSITORY esp-cpp/espp GIT_TAG main SOURCE_SUBDIR lib)
target_link_libraries(myapp PRIVATE espp::espp)

Verification (all passed)

• Configure + --target install to a prefix (no network) → installs libespp_pc.a, ~820 headers, and lib/cmake/espp/{esppConfig,esppConfigVersion,esppTargets,...}.cmake.
• A find_package(espp) consumer linking espp::espp configures, builds, and runs.
• An add_subdirectory(lib) build-tree consumer sees espp::espp and builds/runs (C++23 auto-propagates).
• Existing pc/ test build unchanged; -DESPP_BUILD_PYTHON=ON still builds the wheel as before.

⚠️ Behavioral change to note

The Python module is now opt-in (-DESPP_BUILD_PYTHON=ON, default OFF) so a find_package consumer doesn't drag in pybind11. build.sh/build.ps1 and the CI build_libraries path were updated to pass it, and the wheel/SKBUILD path is unchanged — but this changes what a bare cmake -S lib produces by default.

Caveats

Third-party deps (reflect-cpp, magic_enum, tabulate, fmt, alpaca, cli, csv2, hid-rp, cdr) are vendored submodules — headers installed, objects bundled into libespp_pc.a; only external transitive dep is Threads. Watch for ODR/header clashes if a consumer also uses one of these directly. Requires C++23.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Adds a modern CMake install + export for the host C++ library so external consumers can use a consistent espp::espp target via either find_package(espp) (install-tree) or add_subdirectory(lib) (build-tree), while making Python bindings opt-in by default.

Changes:

• Introduces espp_install_cmake_package() to install/export targets and generate esppConfig.cmake + version files.
• Adds a namespaced build-tree alias (espp::espp) and install-tree export name (EXPORT_NAME espp), plus modern usage requirements (BUILD/INSTALL interface include dirs, PUBLIC C++23 + system libs).
• Gates pybind11 discovery and the Python module build behind ESPP_BUILD_PYTHON (default OFF), updating build scripts to pass -DESPP_BUILD_PYTHON=ON.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
lib/CMakeLists.txt	Adds espp::espp alias + export name, modern install/export wiring, and makes Python bindings opt-in.
lib/espp.cmake	Adds helper to install targets/headers and generate package config + version files.
lib/cmake/esppConfig.cmake.in	New package config template to load exported targets and declare Threads dependency.
lib/build.sh	Forces Python bindings ON for the legacy build script behavior.
lib/build.ps1	Forces Python bindings ON for the legacy build script behavior on Windows.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

✅Static analysis result - no issues found! ✅

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

lib/espp.cmake:225

• Using install(DIRECTORY ...) without FILES_MATCHING/PATTERN filters will copy everything under those include roots into the install prefix (potentially including non-header files like docs, tests, licenses, or build files if any vendored include roots aren’t strictly header-only). To keep installs lean and predictable, restrict this to header patterns (e.g., *.h, *.hpp, *.inl) and/or add explicit excludes for common non-header directories/files.

  foreach(_inc IN LISTS ESPP_INCLUDES ESPP_EXTERNAL_INCLUDES ESPP_EXTERNAL_INCLUDES_SEPARATE)
    install(DIRECTORY ${_inc}/ DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
  endforeach()