Skip to content
Snippets Groups Projects

[Build] Align build to new aidge_core dependencies

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

Merge request reports

Approved by
Code Quality is loading
Test summary results are being parsed

Merged by Grégoire KublerGrégoire Kubler 7 months ago (Sep 20, 2024 9:28am UTC)

Merge details

  • Changes merged into dev with 1b2f2e7f.
  • Deleted the source branch.

Pipeline #55116 failed

Pipeline failed for 1b2f2e7f on dev

Test coverage 80.35% (0.00%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading