stdexec version to use? about senders-io

[maikel]

Indeed. You need to use a specific branch of stdexec that is https://github.com/villevoutilainen/wg21_p2300_std_execution/tree/member-only-customization-points . This branch is an implementation of Villes paper https://wg21.link/p2855

I hoped that the pull request NVIDIA/stdexec#788 would have landed earlier. But it turned out to have a dependency on late customization that Eric currently works on https://wg21.link/p2999

I will try to reflect that in the Readme today

[vsytch]

Thank you, that worked. One more minor issue I hit is incompatibility with catch2 installed by vcpkg.

The catch.hpp header is installed directly to the include/ folder:

vscode ➜ /workspaces/cxx-init/senders-io (main) $ find ${VCPKG_ROOT} -name "catch.hpp"
/opt/vcpkg-2023.08.09/installed/x64-linux/include/catch.hpp

Just needed to fix the include:

diff --git a/tests/net/test_address.cpp b/tests/net/test_address.cpp
index e44ce0b..32349f3 100644
--- a/tests/net/test_address.cpp
+++ b/tests/net/test_address.cpp
@@ -21,7 +21,7 @@
 #include <vector>
 #include <array>
 
-#include <catch2/catch.hpp>
+#include <catch.hpp>
 
 #include "sio/ip/address.hpp"

And the tests failed to link:

[28/30] Linking CXX executable tests/test_sio
FAILED: tests/test_sio 
...
mold: error: undefined symbol: main
>>> referenced by /lib/x86_64-linux-gnu/Scrt1.o:(.text)
>>>               /lib/x86_64-linux-gnu/Scrt1.o:(_start)
c++: error: linker command failed with exit code 1 (use -v to see invocation)

This required changing the catch2 cmake target:

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index 01b70a2..73297b9 100644
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -40,7 +40,7 @@ add_executable(test_sio
   net/test_socket_handle.cpp
 )
 target_include_directories(test_sio PRIVATE ${CMAKE_CURRENT_SOURCE_DIR})
-target_link_libraries(test_sio PRIVATE sio::sio Catch2::Catch2)
+target_link_libraries(test_sio PRIVATE sio::sio Catch2::Catch2WithMain)
 
 include(CTest)
 add_test(NAME test_sio COMMAND test_sio)

[maikel]

I'm probably using a different version of catch2. Thank you for fixing this I will investigate that