[Build] Align build to new aidge_core dependencies
All threads resolved!
All threads resolved!
Context
Simplify dependencies for binding and allow both TEST and PYBIND on. This aligns to the new aidge_core dependencies which are self-sufficient.
This change requires the installed aidge_core library to include the merge request aidge_core!187 (merged)
Modified files
- CmakeLists.txt: remove useless dependencies on pybind and fmt, fix file name of export build file, allow both TEST and PYBIND
- aidge_backend_cpu-config.cmake.in: add missing dependency on aidge_core
- cmake/PybindModuleCreation.cmake: minor fixes
- setup.py: add envvar AIDGE_BUILD_TEST=OFF|ON for building C++ tests
- unit_tests/CMakeLists.txt: minor change, prefer to use PRIVATE when linking final exe/module/library (no behavioral impact)
TODO
-
Need first to merge in aidge_core: aidge_core!187 (merged)
Merge request reports
Activity
Filter activity
- Resolved by Christophe Guillon
@gregkub this is the MR for simplifying C++ modules depending on
aidge_core
, nowaidge_core
brings it's own dependencies. Basically other C++ modules such as this one should be rewritten like this instead of adding cyclic dependencies onPython::Module
etc..
requested review from @gregkub
assigned to @cguillon
mentioned in commit 1b2f2e7f
mentioned in merge request !93 (merged)
Please register or sign in to reply