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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox