debuginfod: Make sure crypto and jsonc are also included in static link

Message ID 20240821133259.1146135-1-mark@klomp.org
State Committed
Headers
Series debuginfod: Make sure crypto and jsonc are also included in static link |

Commit Message

Mark Wielaard Aug. 21, 2024, 1:32 p.m. UTC
  When doing a --enable-gcov build we link all binaries static.
libdebuginfod.so now depends on crypto an jsonc. So also add those
when linking against libdebuginfod.a

      debuginfod/Makefile.am (libdebuginfod): Add $(crypto_LIBS)
      $(jsonc_LIBS) when BUILD_STATIC.

Signed-off-by: Mark Wielaard <mark@klomp.org>
---
 debuginfod/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Mark Wielaard Aug. 21, 2024, 10:38 p.m. UTC | #1
Hi,

On Wed, Aug 21, 2024 at 03:32:59PM +0200, Mark Wielaard wrote:
> When doing a --enable-gcov build we link all binaries static.
> libdebuginfod.so now depends on crypto an jsonc. So also add those
> when linking against libdebuginfod.a
> 
>       debuginfod/Makefile.am (libdebuginfod): Add $(crypto_LIBS)
>       $(jsonc_LIBS) when BUILD_STATIC.

Aaron setup a new fedora coverage try builder. And this patch made
things build.

Pushed.
  

Patch

diff --git a/debuginfod/Makefile.am b/debuginfod/Makefile.am
index e199dc0c74d2..5ad4e188c4c3 100644
--- a/debuginfod/Makefile.am
+++ b/debuginfod/Makefile.am
@@ -47,7 +47,7 @@  libelf = ../libelf/libelf.a -lz
 if DUMMY_LIBDEBUGINFOD
 libdebuginfod = ./libdebuginfod.a
 else
-libdebuginfod = ./libdebuginfod.a -lpthread $(libcurl_LIBS)
+libdebuginfod = ./libdebuginfod.a -lpthread $(libcurl_LIBS) $(crypto_LIBS) $(jsonc_LIBS)
 endif
 else
 libasm = ../libasm/libasm.so