Static builds

[vegas503]

I'm trying to build c-libp2p to be used with static linking. However, when trying to build with...

cmake -S . -B build/ -DCMAKE_POLICY_VERSION_MINIMUM=3.5 -DBUILD_SHARED_LIBS=OFF
make -C build

...most of stuff is built properly, however some tests are failing to link:

[ 95%] Building C object CMakeFiles/test_protocol_noise.dir/tests/protocol/noise/test_protocol_noise.c.o
[ 96%] Linking C executable tests/test_protocol_noise
/usr/bin/ld: lib/libsha3.a(sha3.c.o): in function `shake128_absorb':
sha3.c:(.text+0x206a): multiple definition of `shake128_absorb'; lib/libnoiseprotocol.a(fips202.c.o):fips202.c:(.text+0x1647): first defined here
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/test_protocol_noise.dir/build.make:139: tests/test_protocol_noise] Error 1
make[1]: *** [CMakeFiles/Makefile2:2278: CMakeFiles/test_protocol_noise.dir/all] Error 2
make: *** [Makefile:166: all] Error 2

Versions:

• c-libp2p (latest main)
• gcc (GCC) 15.1.1 20250425
• cmake version 4.0.2-dirty
• GNU Make 4.4.1

I suck at cmake so at this point I'm wondering if there's anything else that needs to be defined from my side, or if c-libp2p is not able to be built as static libs at all?

Let me know if I should provide more details.

Thanks in advance!

[vegas503]

Update - I was able to build it by dropping some tests & binaries:

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9bf2c1d..e7a7282 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -622,7 +622,7 @@ endif()
 add_module(
     protocol_noise
     "src/protocol/noise/protocol_noise.c;src/protocol/noise/protocol_noise_conn.c;src/protocol/noise/protocol_noise_extensions.c;src/protocol/noise/protocol_noise_multiselect.c"
-    tests/protocol/noise/test_protocol_noise.c
+    ""
     ""
     src/protocol/noise
 )
@@ -770,7 +770,7 @@ endif()
 add_module(
     transport_upgrader
     src/transport/upgrader.c
-    tests/transport/test_upgrader.c
+    ""
     ""
     src/transport
 )
@@ -789,65 +789,3 @@ if (TARGET test_transport_upgrader)
     )
     target_include_directories(test_transport_upgrader PRIVATE ${PROJECT_SOURCE_DIR}/lib/libeddsa/lib)
 endif()
-
-# -----------------------------------------------------------------
-# Example dial for identify protocol
-# -----------------------------------------------------------------
-
-add_executable(example_identify_dial
-    examples/example_identify_dial.c
-)
-target_link_libraries(example_identify_dial
-    PRIVATE
-        protocol_tcp
-        protocol_noise
-        protocol_mplex
-        protocol_yamux
-        protocol_identify
-        protocol_handler
-        transport_upgrader
-        multiaddr
-        unsigned_varint
-        peer_id
-        peer_id_secp256k1
-        Threads::Threads
-)
-set_target_properties(example_identify_dial PROPERTIES
-    RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/examples
-)
-
-# On Windows, copy all produced DLLs from bin/ into the examples folder
-if (WIN32 AND BUILD_SHARED_LIBS)
-  add_custom_command(TARGET example_identify_dial
-    POST_BUILD
-    COMMAND ${CMAKE_COMMAND} -E copy_directory
-      ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}
-      $<TARGET_FILE_DIR:example_identify_dial>
-  )
-endif()
-
-# -----------------------------------------------------------------
-# interop binary
-# -----------------------------------------------------------------
-add_executable(interop-c
-    src/interop/interop.c
-)
-target_link_libraries(interop-c
-    PRIVATE
-        protocol_tcp
-        protocol_noise
-        protocol_yamux
-        protocol_mplex
-        protocol_ping
-        transport_upgrader
-        multiaddr
-        peer_id
-        peer_id_ed25519
-        protocol_handler
-        Threads::Threads
-)
-set_target_properties(interop-c PROPERTIES
-    RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}
-)
-
-

I'll see how it goes from here, but something tells me I'm on a wrong path. :)

[uink45]

Hi @vegas503, thanks for raising this!

Can you confirm which OS you are trying to build this on?

We will soon be pushing many changes to the library as a lot of the protocol-related code was rewritten.

For this issue in particular, I will include a patch in those changes.

[vegas503]

@uink45 - thanks for your response!
I'm using Arch. Looking forward to it!