From: Baruch Siach via buildroot <buildroot@buildroot.org>
To: Bernd Kuhls <bernd@kuhls.net>
Cc: buildroot@buildroot.org, Sergio Prado <sergio.prado@e-labworks.com>
Subject: Re: [Buildroot] [PATCH 1/1] package/wolfssl: fix build with libcurl >= 8.19.0 on 32bit archs
Date: Sun, 12 Apr 2026 20:56:02 +0300 [thread overview]
Message-ID: <875x5wdpvx.fsf@tarshish> (raw)
In-Reply-To: <20260411102917.2045764-1-bernd@kuhls.net> (Bernd Kuhls's message of "Sat, 11 Apr 2026 12:29:17 +0200")
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
prev parent reply other threads:[~2026-04-12 17:56 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
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 message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=875x5wdpvx.fsf@tarshish \
--to=buildroot@buildroot.org \
--cc=baruch@tkos.co.il \
--cc=bernd@kuhls.net \
--cc=sergio.prado@e-labworks.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.