[Buildroot] [PATCH 1/1] package/libdrm: link with libatomic

[Buildroot] [PATCH 1/1] package/libdrm: link with libatomic

[Bernd Kuhls]

Link with libatomic if available.

Fixes:
http://autobuild.buildroot.net/results/7ab/7ab583edb135b4b6948fb47e598d1f74e81fea7d/

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
 package/libdrm/libdrm.mk | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/libdrm/libdrm.mk b/package/libdrm/libdrm.mk
index ea956de15b..58d9c34617 100644
--- a/package/libdrm/libdrm.mk
+++ b/package/libdrm/libdrm.mk
@@ -26,6 +26,10 @@ LIBDRM_CFLAGS = $(TARGET_CFLAGS) -DAO_NO_SPARC_V9
 endif
 endif
 
+ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y)
+LIBDRM_LDFLAGS = $(TARGET_LDFLAGS) -latomic
+endif
+
 ifeq ($(BR2_PACKAGE_LIBDRM_INTEL),y)
 LIBDRM_CONF_OPTS += -Dintel=enabled
 LIBDRM_DEPENDENCIES += libpciaccess
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/libdrm: link with libatomic

[Thomas Petazzoni via buildroot]

On Sun, 11 Aug 2024 10:44:24 +0200
Bernd Kuhls <bernd@kuhls.net> wrote:

> Link with libatomic if available.
> 
> Fixes:
> http://autobuild.buildroot.net/results/7ab/7ab583edb135b4b6948fb47e598d1f74e81fea7d/
> 
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> ---
>  package/libdrm/libdrm.mk | 4 ++++
>  1 file changed, 4 insertions(+)

Thanks, but this really should be handled by the libdrm build system
itself. It already has a bunch of logic related to atomic detection.
This really should be improved there. Perhaps report the issue upstream?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot