* build: replace git submodules with CPM.cmake
All 11 submodules (Eigen, fmtlib, msgpack-c, rapidjson, IF97,
REFPROP-headers, multicomplex, Catch2, pybind11, ExcelAddinInstaller,
FindMathematica) are now fetched by CPM.cmake at configure time.
Set CPM_SOURCE_CACHE (e.g. ~/.cache/CPM) to share the download cache
across git worktrees and build directories — no more per-worktree
`git submodule update --init --recursive`.
Vendored deps that have no upstream release cycle (miniz, nlohmann-json,
incbin) remain in externals/ as before.
Source-level changes: angle-bracket includes for rapidjson, IF97, and
REFPROP-headers now that their directories are on the include path via
CPM-provided source dirs rather than relative paths from the repo root.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* fix(python): wire up CPM.cmake in Python wrapper, fix include paths
The Python wrapper CMakeLists.txt was still referencing submodule paths
(externals/Eigen, externals/fmtlib, externals/msgpack-c) removed by the
CPM migration. Include CPM.cmake + dependencies.cmake from the root and
use the ${Pkg_SOURCE_DIR} variables instead. Also adds missing
rapidjson, IF97, and REFPROP_headers include dirs required by CoolProp
sources.
CI workflows drop the now-meaningless `submodules: recursive` checkout
option and add a CPM source-cache step to avoid re-downloading on each
macOS/Windows runner.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: remove git submodule references after CPM.cmake migration
Dependencies are now managed by CPM.cmake (fetched automatically at
CMake configure time), so git submodules no longer exist.
- Drop `--recursive` from all `git clone` commands in Web docs and
wrapper READMEs (26 .rst/.md files)
- Remove `submodules: recursive` from all CI workflow checkout steps
(13 workflow files)
- Remove `git submodule foreach/update` calls from release.bsh and
delete the now-dead pybind11 security-workaround lines
- Drop `--recursive` from build_swigged_matlab.sh and gitMirror.bsh
- Update CONTRIBUTING.md: drop "and its submodules" phrasing
Changelog entries referencing old submodule PRs are left intact as
historical records. The --recursive in buildbot.rst is for the
Dockerfiles repo (unrelated) and is also left unchanged.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* fix(ci): remove empty 'with:' blocks after submodule removal, apply clang-format
After removing 'submodules: recursive' from checkout steps, some workflow
files were left with dangling empty 'with:' blocks that GitHub Actions
rejects as workflow file errors. Remove the empty 'with:' in 9 workflows.
Also apply clang-format to fix spacing in REFPROPMixtureBackend.cpp,
HumidAirProp.cpp, and Helmholtz.cpp.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
* Fix build error: `src/Backends/IF97/IF97Backend.h:54:34: error: call of overloaded ‘abs(double)’ is ambiguous`
Found with the manylinux_2_24_x86_64
gcc (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
* Register MSVC 2019 and 2022 in setup.py
* setup.py: when calling cmake, build in parallel
* Enable using Env variables instead of passing them as args to setup.py
* Github actions for linux: try 1
* use actions/checkout@v3 for submodules
* mod setup.py:; typo
* Random shot for cibuildwheel for all platforms
* I thought package_dir was a flag, but it's positional
* typo in cmake_compiler
* add cython to setup_requires
* try a pryproject.toml to install cython
* try more requirements?
* pywin32 only found on win32 I guess
* Try with CIBW_BEFORE_BUILD instead
* try to enable msvc via vcvarsall on windows, and pass MACOSX_DEPLOYMENT_TARGET
* more tweaks for windoze
* disable tests for now (fails on windows)
* tweak mac again: it seems mac doesn't understand a C++ lambda, so like it's using pre C++11
* tweak
* try 10.15 for mac...
* try to force C++11 since mac picks up the path where lambdas are used...
* Move back down to 10.9 now that C++11 is enabled and it works on mac, it should be enough
* Try to debug win32
* Enable part of the upload step (minus the upload) to list the wheels
* try to allow win32 to fail for now (instead of plain disabling)
* Disable the python_linux.yml workflow, so cibuildwheels works fine.
* Adjust the upload step to point to the right folder
* make LGTM python happy
