All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/frr: add libxcrypt optional dependency
@ 2024-04-07 13:48 Fabrice Fontaine
  2024-04-07 15:39 ` Yann E. MORIN
  0 siblings, 1 reply; 2+ messages in thread
From: Fabrice Fontaine @ 2024-04-07 13:48 UTC (permalink / raw)
  To: buildroot; +Cc: Vadim Kochan, Fabrice Fontaine

Fix the following build failure raised since bump of glibc to version
2.39 in commit b5680f53d60acf8ff6010082f873438a39bd5d97:

/home/autobuild/autobuild/instance-1/output-1/per-package/frr/host/bin/../lib/gcc/i686-buildroot-linux-gnu/12.3.0/../../../../i686-buildroot-linux-gnu/bin/ld: lib/.libs/libfrr.so: undefined reference to `crypt'

Fixes: b5680f53d60acf8ff6010082f873438a39bd5d97
 - http://autobuild.buildroot.org/results/668da3829cacbe883e6cf97ce6191a8180441db6

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/frr/Config.in | 1 +
 package/frr/frr.mk    | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/package/frr/Config.in b/package/frr/Config.in
index ef105202f4..e2ad5d5aa6 100644
--- a/package/frr/Config.in
+++ b/package/frr/Config.in
@@ -9,6 +9,7 @@ config BR2_PACKAGE_FRR
 	depends on BR2_HOST_GCC_AT_LEAST_7 # protobuf-c
 	select BR2_PACKAGE_BASH
 	select BR2_PACKAGE_BUSYBOX_SHOW_OTHERS # bash
+	select BR2_PACKAGE_LIBXCRYPT if BR2_TOOLCHAIN_USES_GLIBC
 	select BR2_PACKAGE_LIBYANG
 	select BR2_PACKAGE_PROTOBUF_C
 	select BR2_PACKAGE_READLINE
diff --git a/package/frr/frr.mk b/package/frr/frr.mk
index 29a9e7b3e8..d20dc07bb7 100644
--- a/package/frr/frr.mk
+++ b/package/frr/frr.mk
@@ -13,7 +13,8 @@ FRR_CPE_ID_PRODUCT = free_range_routing
 FRR_AUTORECONF = YES
 
 FRR_DEPENDENCIES = host-frr readline json-c libyang protobuf-c \
-	$(if $(BR2_PACKAGE_C_ARES),c-ares)
+	$(if $(BR2_PACKAGE_C_ARES),c-ares) \
+	$(if $(BR2_PACKAGE_LIBXCRYPT),libxcrypt)
 
 HOST_FRR_DEPENDENCIES = host-flex host-bison host-elfutils host-python3
 
-- 
2.43.0

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

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/frr: add libxcrypt optional dependency
  2024-04-07 13:48 [Buildroot] [PATCH 1/1] package/frr: add libxcrypt optional dependency Fabrice Fontaine
@ 2024-04-07 15:39 ` Yann E. MORIN
  0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2024-04-07 15:39 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Vadim Kochan, buildroot

fabrice, All,

On 2024-04-07 15:48 +0200, Fabrice Fontaine spake thusly:
> Fix the following build failure raised since bump of glibc to version
> 2.39 in commit b5680f53d60acf8ff6010082f873438a39bd5d97:
> 
> /home/autobuild/autobuild/instance-1/output-1/per-package/frr/host/bin/../lib/gcc/i686-buildroot-linux-gnu/12.3.0/../../../../i686-buildroot-linux-gnu/bin/ld: lib/.libs/libfrr.so: undefined reference to `crypt'
> 
> Fixes: b5680f53d60acf8ff6010082f873438a39bd5d97
>  - http://autobuild.buildroot.org/results/668da3829cacbe883e6cf97ce6191a8180441db6
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
[--SNIP--]
> diff --git a/package/frr/frr.mk b/package/frr/frr.mk
> index 29a9e7b3e8..d20dc07bb7 100644
> --- a/package/frr/frr.mk
> +++ b/package/frr/frr.mk
> @@ -13,7 +13,8 @@ FRR_CPE_ID_PRODUCT = free_range_routing
>  FRR_AUTORECONF = YES
>  
>  FRR_DEPENDENCIES = host-frr readline json-c libyang protobuf-c \

This protobuf-c dependency is not present in master. What branch did you
base this change on?

Otherwise, applied to master, after applying the patch manually because
of the above conflict, thanks.

Regards,
Yann E. MORIN.

> -	$(if $(BR2_PACKAGE_C_ARES),c-ares)
> +	$(if $(BR2_PACKAGE_C_ARES),c-ares) \
> +	$(if $(BR2_PACKAGE_LIBXCRYPT),libxcrypt)
>  
>  HOST_FRR_DEPENDENCIES = host-flex host-bison host-elfutils host-python3
>  
> -- 
> 2.43.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-04-07 15:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-07 13:48 [Buildroot] [PATCH 1/1] package/frr: add libxcrypt optional dependency Fabrice Fontaine
2024-04-07 15:39 ` Yann E. MORIN

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.