From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Bernd Kuhls <bernd@kuhls.net>
Cc: buildroot@buildroot.org,
"James Hilliard" <james.hilliard1@gmail.com>,
"Jérôme Oufella" <jerome.oufella@savoirfairelinux.com>,
"Adrian Perez de Castro" <aperez@igalia.com>,
"Giulio Benetti" <giulio.benetti@benettiengineering.com>
Subject: Re: [Buildroot] [PATCH 1/1] package/icu: needs C++17
Date: Sat, 3 Jan 2026 22:52:32 +0100 [thread overview]
Message-ID: <20260103225232.69a948a6@windsurf> (raw)
In-Reply-To: <20260103154904.1130572-1-bernd@kuhls.net>
Hello Bernd,
On Sat, 3 Jan 2026 16:49:04 +0100
Bernd Kuhls <bernd@kuhls.net> wrote:
> Buildroot commit dcee99507c6767230fd1faeeefe95f5662eb03a3 bumped icu from
> version 73-2 to 77-1.
>
> Upstream raised the minimum C++ requirement to C++17 in version 75-1:
>
> https://github.com/unicode-org/icu/releases/tag/release-75-1
> https://icu.unicode.org/download/75
> "C++ code now requires C++17 [...]"
>
> This commit switches the minimum gcc version to 7.
>
> No autobuilder errors were recorded which relate to the gcc version used.
>
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
Thanks, I did test and indeed the build fails with gcc 6.x, but works
with gcc 7.x, so I've applied, but I had to make quite a few additional
tweaks:
diff --git a/package/gerbera/Config.in b/package/gerbera/Config.in
index 3372d86fba..c5f7cf9cd1 100644
--- a/package/gerbera/Config.in
+++ b/package/gerbera/Config.in
@@ -3,6 +3,7 @@ config BR2_PACKAGE_GERBERA
depends on BR2_USE_MMU # fork()
depends on BR2_TOOLCHAIN_HAS_ATOMIC
depends on BR2_TOOLCHAIN_HAS_THREADS
+ depends on BR2_HOST_GCC_AT_LEAST_7 # icu
depends on BR2_TOOLCHAIN_GCC_AT_LEAST_8 # C++17 filesystem
depends on BR2_INSTALL_LIBSTDCPP
depends on !BR2_STATIC_LIBS
@@ -28,9 +29,10 @@ config BR2_PACKAGE_GERBERA
https://gerbera.io
-comment "gerbera needs a toolchain w/ C++, dynamic library, threads, wchar, gcc >= 8"
+comment "gerbera needs a toolchain w/ C++, dynamic library, threads, wchar, gcc >= 8, host gcc >= 7"
depends on BR2_USE_MMU
depends on BR2_TOOLCHAIN_HAS_ATOMIC
depends on !BR2_INSTALL_LIBSTDCPP || BR2_STATIC_LIBS || \
!BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR || \
- !BR2_TOOLCHAIN_GCC_AT_LEAST_8
+ !BR2_TOOLCHAIN_GCC_AT_LEAST_8 || \
+ !BR2_HOST_GCC_AT_LEAST_7
diff --git a/package/icu/Config.in b/package/icu/Config.in
index 542acc4187..f5686085dd 100644
--- a/package/icu/Config.in
+++ b/package/icu/Config.in
@@ -3,7 +3,7 @@ config BR2_PACKAGE_ICU
depends on BR2_INSTALL_LIBSTDCPP
depends on BR2_USE_WCHAR
depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
- depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
+ depends on BR2_HOST_GCC_AT_LEAST_7 # host-icu
depends on BR2_TOOLCHAIN_HAS_THREADS
# icu does some funky things by generating by itself an ELF
# file, and it cannot easily be changed to generate FLAT
diff --git a/package/qt5/qt5webengine/Config.in b/package/qt5/qt5webengine/Config.in
index 0dd1483c57..2c16d819d2 100644
--- a/package/qt5/qt5webengine/Config.in
+++ b/package/qt5/qt5webengine/Config.in
@@ -10,10 +10,11 @@ config BR2_PACKAGE_QT5WEBENGINE_ARCH_SUPPORTS
depends on !BR2_BINFMT_FLAT # qt5base-icu
depends on BR2_USE_MMU # libglib2, qt5base-dbus
-comment "qt5webengine needs udev /dev management and a glibc toolchain w/ host gcc >= 7, threads, wchar"
+comment "qt5webengine needs udev /dev management and a glibc toolchain w/ gcc >= 7, threads, wchar, host gcc >= 7"
depends on BR2_PACKAGE_QT5WEBENGINE_ARCH_SUPPORTS
depends on !BR2_PACKAGE_HAS_UDEV || !BR2_TOOLCHAIN_USES_GLIBC || \
!BR2_HOST_GCC_AT_LEAST_7 || \
+ !BR2_TOOLCHAIN_GCC_AT_LEAST_7 || \
!BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR
comment "qt5webengine needs an OpenGL and EGL-capable backend"
@@ -24,6 +25,7 @@ config BR2_PACKAGE_QT5WEBENGINE
bool "qt5webengine"
depends on BR2_PACKAGE_QT5WEBENGINE_ARCH_SUPPORTS
depends on BR2_TOOLCHAIN_USES_GLIBC # execinfo.h, mallinfo
+ depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # qt5base-icu
depends on BR2_HOST_GCC_AT_LEAST_7 # qt5base-icu
depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # libvpx
depends on BR2_USE_WCHAR # libglib2
Thanks a lot for having looked into this, much appreciated!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2026-01-03 21:52 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-03 15:49 [Buildroot] [PATCH 1/1] package/icu: needs C++17 Bernd Kuhls
2026-01-03 21:52 ` Thomas Petazzoni via buildroot [this message]
2026-01-13 19:48 ` Arnout Vandecappelle via buildroot
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=20260103225232.69a948a6@windsurf \
--to=buildroot@buildroot.org \
--cc=aperez@igalia.com \
--cc=bernd@kuhls.net \
--cc=giulio.benetti@benettiengineering.com \
--cc=james.hilliard1@gmail.com \
--cc=jerome.oufella@savoirfairelinux.com \
--cc=thomas.petazzoni@bootlin.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.