Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[ci skip] Knutel/peter d cmake include fixes (#397)
Overview -------- Overhaul of MAP build system and included proper support for installation Updated Documentation Updated/Cleaned up Argos and simplified build system Various bug fixes Details ------- * simpler boost detection on mac * cmake clean up: target specific include paths * boost 1.76 * boost 1.72 specific magic no longer needed * Moved reliance on include paths back to sparta.cmake file * Made simdb include a private include * PUBLIC includes for Sparta and SimDB to pass to regression * cmake file to enable find_package(Sparta) in other projects * make sure core setup.py actually uses the interpreter found by find_package(PythonInterp) * more modern way to find python3, hopefully also robust in conda/venv * cleaner cmake for testing macros for simdb and sparta * helios build now using FindSparta.cmake * pipeviewer cannot use find_package(sparta), which only works once installed * installing all py files and libs but module paths don't work yet after install * simpler setup for cython build - working for transactiondb, rest todo * cleaner HDF5 libs config for sparta * Documentation cleanup * Major overhaul of how argos ties into cmake * cmake pushes all config into setuptools via setup.cfg where possible (such that all found libs in cmake and sparta deps are automatically in sync with rest of the build) * python setup.py then manages the cythonized build *and* installation of all python files by doing python setup.py install * Cleaned up all python imports so they are relative imports (previously mixed absolute vs relative which breaks when you install the package) * relocated some of the cython files in the source tree for easier build and install * Fixed static method calls w/o object * Move __is_mac_os and __gen_message outside of the ShortcutHelp class * Clean up flake8 errors * make sure setuptools and cython use the right C/CXX/LD * proper dependencies for setup.py * gui script vs console script: should properly call pythonw * build argos_dumper again * _ vs __ otherwise cannot find the free helper functions in the class * Move sparta library target towards the end of sparta/CMakeLists.txt so it picks up all of the compile/link flags Bump required Boost version down to 1.74 so we can build on default Ubuntu environment * Pass CMAKE_INSTALL_PREFIX to setup.py install - Enables installing package to a non-default directory * Add Github workflow to run linters on Argos Reorganize mypy stubs and .ini locations to work with relative imports in Argos Use pip instead of (deprecated) setup.py install to install Argos - Also properly handle non-default install prefix * Set a a CMAKE_MACRO file path for the testers; allows sparta build anywhere * Update README.md for Conda on MacOS * seperated build for sparta and helios folder * Updated README and build rules for new layout * Fixed bad dir path in regression * Renamed DatabaseDump tool * Added missing rt library for Linux builds * Put in check for CPP check module * Fixed new compilation issue with HDF test * ANOTHER HDF5 test fix * Forced HDF5 test to be C++ testing * Removed rt library link * ONE more tweak: use CXX version of HDF5 * Fixed formatting * Updated README with install directions --------- Co-authored-by: Peter Debacker <[email protected]> Co-authored-by: Brett Dutro <[email protected]> Co-authored-by: Benoy Alexander <[email protected]> 4033e18
- Loading branch information