Sync build configuration with Memray

- Start setting `CMAKE_CXX_VISIBILITY_PRESET` to `hidden` to avoid
  unintentionally exposing symbols from our extension module that could
  potentially conflict with other modules.
- Ensure that directories searched for shared libraries are added to the
  RPATH with `CMAKE_INSTALL_RPATH_USE_LINK_PATH`.
- Ensure that the Python version we find includes a configuration for
  building extension modules that against the stable ABI.
- Build with -Wall to enable extra warnings.
- Pass `-D_FILE_OFFSET_BITS=64` to opt into 64-bit file IO APIs even on
  32-bit platforms. This currently doesn't have any effect, since we
  only build for 64-bit platforms, but it would be needed if we ever
  added i686 support, and matching Memray will make maintenance cheaper.
- Enable extra debugging in the C++ STL for debug builds.
- Build with link time optimizations enabled for CMake release mode
  builds.
- Optimize for debugging in CMake debug builds.
- Require `scikit-build-core` 0.9 or newer.
- Enable extra logging during builds.
- Avoid stripping debug info from the built extension module.
- Automatically build in debug mode rather than release mode when
  passing the `-e` / `--editable` flag to `pip install`.

Signed-off-by: Matt Wozniski <godlygeek@gmail.com>

Author: godlygeek

CMakeLists.txt

@@ -5,9 +5,24 @@ project(pystack LANGUAGES CXX)
 set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
+set(CMAKE_CXX_VISIBILITY_PRESET hidden)
+set(CMAKE_INSTALL_RPATH_USE_LINK_PATH TRUE)
 
 # Find Python
-find_package(Python 3.9 COMPONENTS Interpreter Development.Module REQUIRED)
+find_package(Python COMPONENTS Interpreter Development.Module Development.SABIModule REQUIRED)
+
+# Set compiler flags
+add_compile_options(-Wall)
+add_compile_definitions(_FILE_OFFSET_BITS=64)
+
+add_compile_definitions(
+    "$<$<CONFIG:Debug>:_GLIBCXX_DEBUG>"
+    "$<$<CONFIG:Debug>:_LIBCPP_DEBUG>"
+)
+
+set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -flto")
+set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "${CMAKE_CXX_FLAGS_RELWITHDEBINFO} -flto")
+set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -Og")
 
 # Find nanobind
 execute_process(

pyproject.toml

@@ -58,6 +58,11 @@ Homepage = "https://github.com/bloomberg/pystack"
 pystack = "pystack.__main__:main"
 
 [tool.scikit-build]
+minimum-version = "0.9"
+cmake.verbose = true
+cmake.build-type = "RelWithDebInfo"
+logging.level = "INFO"
+install.strip = false
 wheel.packages = ["src/pystack"]
 wheel.install-dir = "pystack"
 wheel.exclude = ["pystack/_pystack/**"]
@@ -75,8 +80,11 @@ sdist.exclude = [
   "uv.lock",
 ]
 
-[tool.scikit-build.cmake.define]
-CMAKE_BUILD_TYPE = "Release"
+[[tool.scikit-build.overrides]]
+if.state = "editable"
+build-dir = "build"
+editable.verbose = false
+cmake.build-type = "Debug"
 
 [tool.ruff]
 line-length = 95