Skip to content

Commit e519f79

Browse files
committed
fix: Remove unnecessary wsock32 dependency on Windows
Looks like it was added by a mistake: 1. We didn't have this dependency before, in fact, autotools still doesn't have it, it was added only to CMake. 2. During my testing, when building the echo_bot example, there are no unresolved symbols left after linking against `libtox.a libsodium.a -pthread -lws2_32 -liphlpapi`, which indicates that `-lwsock32` is unnecessary.
1 parent ed2b60c commit e519f79

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,7 @@ elseif(TARGET Threads::Threads)
454454
set(toxcore_LINK_LIBRARIES ${toxcore_LINK_LIBRARIES} Threads::Threads)
455455
endif()
456456
if(WIN32)
457-
set(toxcore_LINK_LIBRARIES ${toxcore_LINK_LIBRARIES} iphlpapi wsock32 ws2_32)
457+
set(toxcore_LINK_LIBRARIES ${toxcore_LINK_LIBRARIES} iphlpapi ws2_32)
458458
endif()
459459

460460
################################################################################

0 commit comments

Comments
 (0)