All of lore.kernel.org
 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 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.