Merge branch 'fix_fmt_catch2' into 'dev'

Change fmt library to system and upgrade Catch2

See merge request !225

CMakeLists.txt

@@ -115,9 +115,12 @@ if (PYBIND)
     include(PybindDependency)
     add_pybind_dependency(${module_name})
     ##
+
+    target_link_libraries(${pybind_module_name} PRIVATE fmt::fmt)
 endif()
 
-target_link_libraries(${module_name} PUBLIC Threads::Threads fmt::fmt)
+target_link_libraries(${module_name} PRIVATE fmt::fmt)
+target_link_libraries(${module_name} PUBLIC Threads::Threads)
 target_compile_features(${module_name} PRIVATE cxx_std_14)
 
 if (DOSANITIZE STREQUAL "ON")

include/aidge/data/Elts.hpp

@@ -101,12 +101,12 @@ private:
 template<>
 struct fmt::formatter<Aidge::Elts_t> {
     template<typename ParseContext>
-    inline constexpr auto parse(ParseContext& ctx) {
+    inline constexpr auto parse(ParseContext& ctx) -> decltype(ctx.begin()) {
         return ctx.begin();
     }
 
     template<typename FormatContext>
-    inline auto format(Aidge::Elts_t const& elt, FormatContext& ctx) {
+    inline auto format(const Aidge::Elts_t& elt, FormatContext& ctx) const {
         return fmt::format_to(ctx.out(), "{}:{}", elt.data, elt.token);
     }
 };

unit_tests/CMakeLists.txt

@@ -63,6 +63,7 @@ endif()
 target_link_libraries(tests${module_name} PRIVATE ${module_name})
 
 target_link_libraries(tests${module_name} PRIVATE Catch2::Catch2WithMain)
+target_link_libraries(tests${module_name} PRIVATE fmt::fmt)
 
 list(APPEND CMAKE_MODULE_PATH ${catch2_SOURCE_DIR}/extras)
 include(CTest)