diff mbox series

[1/1] package/freeradius-server: fix libkrb5 build

Message ID 20220814220241.203440-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/freeradius-server: fix libkrb5 build | expand

Commit Message

Fabrice Fontaine Aug. 14, 2022, 10:02 p.m. UTC
Fix the following libkrb5 build failure raised since the addition of the
package in commit 736c4c1655d93652a9a7e79235bbe726b3d46176:

configure: error: in `/home/giuliobenetti/autobuild/run/instance-2/output-1/build/freeradius-server-3.2.0/src/modules/rlm_krb5':
configure: error: cannot run test program while cross compiling

Fixes:
 - http://autobuild.buildroot.org/results/3968d611458060480d341bd60f7c50209a0ace15

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/freeradius-server/freeradius-server.mk | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Yann E. MORIN Aug. 15, 2022, 8:03 p.m. UTC | #1
Fabrice, All,

On 2022-08-15 00:02 +0200, Fabrice Fontaine spake thusly:
> Fix the following libkrb5 build failure raised since the addition of the
> package in commit 736c4c1655d93652a9a7e79235bbe726b3d46176:
> 
> configure: error: in `/home/giuliobenetti/autobuild/run/instance-2/output-1/build/freeradius-server-3.2.0/src/modules/rlm_krb5':
> configure: error: cannot run test program while cross compiling
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/3968d611458060480d341bd60f7c50209a0ace15
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

For the sake of sharing the blame with whomever applies that patch:

Acked-by: Yann E. MORIN <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/freeradius-server/freeradius-server.mk | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/package/freeradius-server/freeradius-server.mk b/package/freeradius-server/freeradius-server.mk
> index 2f4c118da5..5b74a3b01a 100644
> --- a/package/freeradius-server/freeradius-server.mk
> +++ b/package/freeradius-server/freeradius-server.mk
> @@ -15,6 +15,12 @@ FREERADIUS_SERVER_CPE_ID_PRODUCT = freeradius
>  FREERADIUS_SERVER_DEPENDENCIES = libtalloc
>  FREERADIUS_SERVER_AUTORECONF = YES
>  
> +# We're patching src/modules/rlm_krb5/configure.ac
> +define FREERADIUS_SERVER_RUN_KRB5_AUTOCONF
> +	cd $(@D)/src/modules/rlm_krb5; $(AUTOCONF) -I$(@D)
> +endef
> +FREERADIUS_SERVER_PRE_CONFIGURE_HOOKS += FREERADIUS_SERVER_RUN_KRB5_AUTOCONF
> +
>  # some compiler checks are not supported while cross compiling.
>  # instead of removing those checks, we cache the answers
>  FREERADIUS_SERVER_CONF_OPTS += \
> -- 
> 2.35.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Thomas Petazzoni Aug. 15, 2022, 8:21 p.m. UTC | #2
On Mon, 15 Aug 2022 00:02:41 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Fix the following libkrb5 build failure raised since the addition of the
> package in commit 736c4c1655d93652a9a7e79235bbe726b3d46176:
> 
> configure: error: in `/home/giuliobenetti/autobuild/run/instance-2/output-1/build/freeradius-server-3.2.0/src/modules/rlm_krb5':
> configure: error: cannot run test program while cross compiling
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/3968d611458060480d341bd60f7c50209a0ace15
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/freeradius-server/freeradius-server.mk | 6 ++++++
>  1 file changed, 6 insertions(+)

Applied to master, thanks.

Thomas
diff mbox series

Patch

diff --git a/package/freeradius-server/freeradius-server.mk b/package/freeradius-server/freeradius-server.mk
index 2f4c118da5..5b74a3b01a 100644
--- a/package/freeradius-server/freeradius-server.mk
+++ b/package/freeradius-server/freeradius-server.mk
@@ -15,6 +15,12 @@  FREERADIUS_SERVER_CPE_ID_PRODUCT = freeradius
 FREERADIUS_SERVER_DEPENDENCIES = libtalloc
 FREERADIUS_SERVER_AUTORECONF = YES
 
+# We're patching src/modules/rlm_krb5/configure.ac
+define FREERADIUS_SERVER_RUN_KRB5_AUTOCONF
+	cd $(@D)/src/modules/rlm_krb5; $(AUTOCONF) -I$(@D)
+endef
+FREERADIUS_SERVER_PRE_CONFIGURE_HOOKS += FREERADIUS_SERVER_RUN_KRB5_AUTOCONF
+
 # some compiler checks are not supported while cross compiling.
 # instead of removing those checks, we cache the answers
 FREERADIUS_SERVER_CONF_OPTS += \