public inbox for buildroot@busybox.net
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/wolfssl: fix build with libcurl >= 8.19.0 on 32bit archs
@ 2026-04-11 10:29 Bernd Kuhls
  2026-04-12 17:56 ` Baruch Siach via buildroot
  0 siblings, 1 reply; 2+ messages in thread
From: Bernd Kuhls @ 2026-04-11 10:29 UTC (permalink / raw)
  To: buildroot; +Cc: Sergio Prado

wolfssl requires definition of SIZEOF_LONG_LONG. Commit 3a5e071e4f9
("package/libcurl: security bump to version 8.19.0") removed
SIZEOF_LONG_LONG definitions from libcurl autotools build system,
breaking build with wolfssl.

Fixes:
https://autobuild.buildroot.net/results/baa/baa74a47464ed3e21d0adad64dd0c7c708c9ef85/

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
Please note that Baruch sent a fix to libcurl:
https://patchwork.ozlabs.org/project/buildroot/patch/ec612e55099118043d351762107b9c1a18387130.1775471071.git.baruch@tkos.co.il/

although the problem should be fixed in wolfSSL instead, see
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=294287
https://github.com/wolfSSL/wolfssl/issues/9936
for details.

 ...TS_H-in-options.h-rather-than-config.patch | 60 +++++++++++++++++++
 1 file changed, 60 insertions(+)
 create mode 100644 package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch

diff --git a/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch b/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
new file mode 100644
index 0000000000..67f9672c2a
--- /dev/null
+++ b/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
@@ -0,0 +1,60 @@
+From b54173fc402c30e81d66b84e2f9725cb6d944b40 Mon Sep 17 00:00:00 2001
+From: Kareem <kareem@wolfssl.com>
+Date: Fri, 27 Mar 2026 17:01:02 -0700
+Subject: [PATCH] Define HAVE_LIMITS_H in options.h rather than config.h since
+ types.h depends on this definition and config.h isn't consistently available
+ at runtime. Fixes #9936.
+
+Upstream: https://github.com/wolfSSL/wolfssl/pull/10097
+
+[Bernd: rebased for 5.7.2]
+Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
+---
+ cmake/config.in    | 3 ---
+ cmake/options.h.in | 3 +++
+ configure.ac       | 4 +++-
+ 3 files changed, 6 insertions(+), 4 deletions(-)
+
+diff --git a/cmake/config.in b/cmake/config.in
+index f2524e41e43..6054b6dbe7b 100644
+--- a/cmake/config.in
++++ b/cmake/config.in
+@@ -19,9 +19,6 @@
+ /* Define to 1 if you have the `gmtime_r' function. */
+ #cmakedefine HAVE_GMTIME_R @HAVE_GMTIME_R@
+ 
+-/* Define to 1 if you have the <limits.h> header file. */
+-#cmakedefine HAVE_LIMITS_H @HAVE_LIMITS_H@
+-
+ /* Define to 1 if you have the <pcap/pcap.h> header file. */
+ #cmakedefine HAVE_PCAP_PCAP_H @HAVE_PCAP_PCAP_H@
+ 
+diff --git a/cmake/options.h.in b/cmake/options.h.in
+index 985b54241d6..c46ada8045d 100644
+--- a/cmake/options.h.in
++++ b/cmake/options.h.in
+@@ -31,6 +31,9 @@ extern "C" {
+ #endif
+ 
+ #ifndef WOLFSSL_OPTIONS_IGNORE_SYS
++/* Since types.h depends on HAVE_LIMITS_H, we must define it in options.h. */
++#undef HAVE_LIMITS_H
++#cmakedefine HAVE_LIMITS_H @HAVE_LIMITS_H@
+ #undef _GNU_SOURCE
+ #cmakedefine _GNU_SOURCE
+ #undef _POSIX_THREADS
+diff --git a/configure.ac b/configure.ac
+index bce600f2ecc..c5cf9f320c7 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -116,7 +116,9 @@
+     AM_CCASFLAGS="$AM_CCASFLAGS -DWOLFSSL_EXPERIMENTAL_SETTINGS"
+ fi
+ 
+-AC_CHECK_HEADERS([arpa/inet.h fcntl.h limits.h netdb.h netinet/in.h stddef.h time.h sys/ioctl.h sys/socket.h sys/time.h errno.h sys/un.h])
++AC_CHECK_HEADERS([arpa/inet.h fcntl.h netdb.h netinet/in.h stddef.h time.h sys/ioctl.h sys/socket.h sys/time.h errno.h sys/un.h])
++# Special case: Since types.h depends on HAVE_LIMITS_H, we must define it in options.h.
++AC_CHECK_HEADER([limits.h], [AM_CPPFLAGS="$AM_CPPFLAGS -DHAVE_LIMITS_H=1"], [])
+ AC_CHECK_LIB([network],[socket])
+ AC_C_BIGENDIAN
+ AC_C___ATOMIC
-- 
2.47.3

_______________________________________________
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/wolfssl: fix build with libcurl >= 8.19.0 on 32bit archs
  2026-04-11 10:29 [Buildroot] [PATCH 1/1] package/wolfssl: fix build with libcurl >= 8.19.0 on 32bit archs Bernd Kuhls
@ 2026-04-12 17:56 ` Baruch Siach via buildroot
  0 siblings, 0 replies; 2+ messages in thread
From: Baruch Siach via buildroot @ 2026-04-12 17:56 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: buildroot, Sergio Prado

Hi Bernd,

On Sat, Apr 11 2026, Bernd Kuhls wrote:
> wolfssl requires definition of SIZEOF_LONG_LONG. Commit 3a5e071e4f9
> ("package/libcurl: security bump to version 8.19.0") removed
> SIZEOF_LONG_LONG definitions from libcurl autotools build system,
> breaking build with wolfssl.
>
> Fixes:
> https://autobuild.buildroot.net/results/baa/baa74a47464ed3e21d0adad64dd0c7c708c9ef85/
>
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> ---
> Please note that Baruch sent a fix to libcurl:
> https://patchwork.ozlabs.org/project/buildroot/patch/ec612e55099118043d351762107b9c1a18387130.1775471071.git.baruch@tkos.co.il/

Your suggestion is better than mine for Buildroot, IMO.

Tested-by: Baruch Siach <baruch@tkos.co.il>

Thanks,
baruch

> although the problem should be fixed in wolfSSL instead, see
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=294287
> https://github.com/wolfSSL/wolfssl/issues/9936
> for details.
>
>  ...TS_H-in-options.h-rather-than-config.patch | 60 +++++++++++++++++++
>  1 file changed, 60 insertions(+)
>  create mode 100644 package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
>
> diff --git
> a/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
> b/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
> new file mode 100644
> index 0000000000..67f9672c2a
> --- /dev/null
> +++ b/package/wolfssl/0001-Define-HAVE_LIMITS_H-in-options.h-rather-than-config.patch
> @@ -0,0 +1,60 @@
> +From b54173fc402c30e81d66b84e2f9725cb6d944b40 Mon Sep 17 00:00:00 2001
> +From: Kareem <kareem@wolfssl.com>
> +Date: Fri, 27 Mar 2026 17:01:02 -0700
> +Subject: [PATCH] Define HAVE_LIMITS_H in options.h rather than config.h since
> + types.h depends on this definition and config.h isn't consistently available
> + at runtime. Fixes #9936.
> +
> +Upstream: https://github.com/wolfSSL/wolfssl/pull/10097
> +
> +[Bernd: rebased for 5.7.2]
> +Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> +---
> + cmake/config.in    | 3 ---
> + cmake/options.h.in | 3 +++
> + configure.ac       | 4 +++-
> + 3 files changed, 6 insertions(+), 4 deletions(-)
> +
> +diff --git a/cmake/config.in b/cmake/config.in
> +index f2524e41e43..6054b6dbe7b 100644
> +--- a/cmake/config.in
> ++++ b/cmake/config.in
> +@@ -19,9 +19,6 @@
> + /* Define to 1 if you have the `gmtime_r' function. */
> + #cmakedefine HAVE_GMTIME_R @HAVE_GMTIME_R@
> + 
> +-/* Define to 1 if you have the <limits.h> header file. */
> +-#cmakedefine HAVE_LIMITS_H @HAVE_LIMITS_H@
> +-
> + /* Define to 1 if you have the <pcap/pcap.h> header file. */
> + #cmakedefine HAVE_PCAP_PCAP_H @HAVE_PCAP_PCAP_H@
> + 
> +diff --git a/cmake/options.h.in b/cmake/options.h.in
> +index 985b54241d6..c46ada8045d 100644
> +--- a/cmake/options.h.in
> ++++ b/cmake/options.h.in
> +@@ -31,6 +31,9 @@ extern "C" {
> + #endif
> + 
> + #ifndef WOLFSSL_OPTIONS_IGNORE_SYS
> ++/* Since types.h depends on HAVE_LIMITS_H, we must define it in options.h. */
> ++#undef HAVE_LIMITS_H
> ++#cmakedefine HAVE_LIMITS_H @HAVE_LIMITS_H@
> + #undef _GNU_SOURCE
> + #cmakedefine _GNU_SOURCE
> + #undef _POSIX_THREADS
> +diff --git a/configure.ac b/configure.ac
> +index bce600f2ecc..c5cf9f320c7 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -116,7 +116,9 @@
> +     AM_CCASFLAGS="$AM_CCASFLAGS -DWOLFSSL_EXPERIMENTAL_SETTINGS"
> + fi
> + 
> +-AC_CHECK_HEADERS([arpa/inet.h fcntl.h limits.h netdb.h netinet/in.h stddef.h
> time.h sys/ioctl.h sys/socket.h sys/time.h errno.h sys/un.h])
> ++AC_CHECK_HEADERS([arpa/inet.h fcntl.h netdb.h netinet/in.h stddef.h time.h
> sys/ioctl.h sys/socket.h sys/time.h errno.h sys/un.h])
> ++# Special case: Since types.h depends on HAVE_LIMITS_H, we must define it in options.h.
> ++AC_CHECK_HEADER([limits.h], [AM_CPPFLAGS="$AM_CPPFLAGS -DHAVE_LIMITS_H=1"], [])
> + AC_CHECK_LIB([network],[socket])
> + AC_C_BIGENDIAN
> + AC_C___ATOMIC

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
_______________________________________________
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:[~2026-04-12 17:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-11 10:29 [Buildroot] [PATCH 1/1] package/wolfssl: fix build with libcurl >= 8.19.0 on 32bit archs Bernd Kuhls
2026-04-12 17:56 ` Baruch Siach via buildroot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox