All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0
@ 2020-04-18 22:59 Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0 Peter Seiderer
                   ` (6 more replies)
  0 siblings, 7 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As a side effect of avoiding gcc bug 57694 ([1]) through commit
'GCC: revoke constexpr before 5.0' ([2]) for qtbase the minimum
gcc requirement was lifted to 5.0 ([3], [4]), so bump the
buildroot determined one too.

Adjust the two packages (pinentry, wireshark) which are brave enough
to do a 'select BR2_PACKAGE_QT5' accordingly.

Fixes:
  http://autobuild.buildroot.net/results/74f5249d1ee14c1269685af53a65055da7dc2ee6

  ../../include/QtCore/../../src/corelib/thread/qbasicatomic.h:61:4: error: #error "Qt requires C++11 support"
   #  error "Qt requires C++11 support"

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57694
[2] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=c61cedcc5475a77fb94ed12788f61039835f8079
[3] https://codereview.qt-project.org/c/qt/qtbase/+/285582
[4] https://bugreports.qt.io/browse/QTBUG-81761

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/pinentry/Config.in  | 6 +++---
 package/qt5/Config.in       | 6 +++---
 package/wireshark/Config.in | 6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/package/pinentry/Config.in b/package/pinentry/Config.in
index ef70908502..23f532e2e6 100644
--- a/package/pinentry/Config.in
+++ b/package/pinentry/Config.in
@@ -61,7 +61,7 @@ config BR2_PACKAGE_PINENTRY_QT5
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_USE_WCHAR
 	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_5_0
 	depends on !BR2_arc
 	depends on !BR2_STATIC_LIBS
 	select BR2_PACKAGE_QT5
@@ -70,10 +70,10 @@ config BR2_PACKAGE_PINENTRY_QT5
 	help
 	  The pinentry-qt5 tool
 
-comment "pinentry-qt5 needs a toolchain w/ wchar, NPTL, gcc >= 4.8, C++, dynamic library"
+comment "pinentry-qt5 needs a toolchain w/ wchar, NPTL, gcc >= 5.0, C++, dynamic library"
 	depends on !BR2_arc
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
-		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
+		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_5_0 || \
 		BR2_STATIC_LIBS
 
 endif
diff --git a/package/qt5/Config.in b/package/qt5/Config.in
index 77dce16038..cd9c93735d 100644
--- a/package/qt5/Config.in
+++ b/package/qt5/Config.in
@@ -11,19 +11,19 @@ config BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	# ARM needs BLX, so v5t+
 	depends on !BR2_ARM_CPU_ARMV4
 
-comment "Qt5 needs a toolchain w/ gcc >= 4.8, wchar, NPTL, C++, dynamic library"
+comment "Qt5 needs a toolchain w/ gcc >= 5.0, wchar, NPTL, C++, dynamic library"
 	depends on !BR2_ARM_CPU_ARMV4
 	depends on !BR2_arc
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
 		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS || \
-		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+		!BR2_TOOLCHAIN_GCC_AT_LEAST_5_0
 
 menuconfig BR2_PACKAGE_QT5
 	bool "Qt5"
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_USE_WCHAR
 	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # C++11
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_5_0 # C++11, GCC_BUG_57694
 	depends on !BR2_ARM_CPU_ARMV4 # needs ARMv5+
 	# no built-in double-conversion support
 	depends on !BR2_arc
diff --git a/package/wireshark/Config.in b/package/wireshark/Config.in
index e52edb9f69..f55cfeb497 100644
--- a/package/wireshark/Config.in
+++ b/package/wireshark/Config.in
@@ -21,7 +21,7 @@ config BR2_PACKAGE_WIRESHARK_GUI
 	default y
 	depends on BR2_INSTALL_LIBSTDCPP # qt5
 	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # qt5
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # qt5
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_5_0 # qt5
 	depends on !BR2_arc # qt5
 	depends on BR2_PACKAGE_HAS_UDEV # qt5 input
 	select BR2_PACKAGE_QT5
@@ -36,10 +36,10 @@ config BR2_PACKAGE_WIRESHARK_GUI
 	help
 	  Enable Qt5 GUI
 
-comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
+comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 5.0"
 	depends on !BR2_arc
 	depends on !BR2_PACKAGE_HAS_UDEV || !BR2_INSTALL_LIBSTDCPP || \
-		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_5_0
 
 endif # BR2_PACKAGE_WIRESHARK
 
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
@ 2020-04-18 22:59 ` Peter Seiderer
  2020-04-19 13:03   ` Thomas Petazzoni
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 3/6] package/qt5tools: " Peter Seiderer
                   ` (5 subsequent siblings)
  6 siblings, 1 reply; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/qt5/qt5base/Config.in | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/package/qt5/qt5base/Config.in b/package/qt5/qt5base/Config.in
index 60a611e292..358a54bc1d 100644
--- a/package/qt5/qt5base/Config.in
+++ b/package/qt5/qt5base/Config.in
@@ -233,9 +233,7 @@ config BR2_PACKAGE_QT5BASE_FONTCONFIG
 
 config BR2_PACKAGE_QT5BASE_HARFBUZZ
 	bool "harfbuzz support"
-	select BR2_PACKAGE_HARFBUZZ if \
-		BR2_TOOLCHAIN_HAS_SYNC_4 && \
-		BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+	select BR2_PACKAGE_HARFBUZZ if BR2_TOOLCHAIN_HAS_SYNC_4
 	help
 	  This option enables HarfBuzz support (either system harfbuzz
 	  if the toolchain supports __sync for 4 bytes, or the qt
@@ -273,16 +271,11 @@ config BR2_PACKAGE_QT5BASE_DBUS
 config BR2_PACKAGE_QT5BASE_ICU
 	bool "Enable ICU support"
 	depends on !BR2_BINFMT_FLAT # icu
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # icu
 	select BR2_PACKAGE_ICU
 	help
 	  This option enables ICU support in Qt5. This is for example
 	  needed for Qt5Webkit.
 
-comment "icu support needs a toolchain w/ gcc >= 4.8"
-	depends on !BR2_BINFMT_FLAT
-	depends on !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
-
 config BR2_PACKAGE_QT5BASE_TSLIB
 	bool "Enable Tslib support"
 	select BR2_PACKAGE_TSLIB
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 3/6] package/qt5tools: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0 Peter Seiderer
@ 2020-04-18 22:59 ` Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 4/6] package/qt5webengine: " Peter Seiderer
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/qt5/qt5tools/Config.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/package/qt5/qt5tools/Config.in b/package/qt5/qt5tools/Config.in
index 60106c1ee3..f01b51d2e8 100644
--- a/package/qt5/qt5tools/Config.in
+++ b/package/qt5/qt5tools/Config.in
@@ -23,7 +23,6 @@ config BR2_PACKAGE_QT5TOOLS_QDOC_TOOL
 	# currently supported in Buildroot, unless the target is
 	# already supported by llvm.
 	depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS # llvm, clang
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # llvm, clang
 	depends on BR2_TOOLCHAIN_HAS_THREADS # llvm, clang
 	depends on BR2_INSTALL_LIBSTDCPP # llvm, clang
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # llvm, clang
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 4/6] package/qt5webengine: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0 Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 3/6] package/qt5tools: " Peter Seiderer
@ 2020-04-18 22:59 ` Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 5/6] package/qt5webkit: " Peter Seiderer
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/qt5/qt5webengine/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/qt5/qt5webengine/Config.in b/package/qt5/qt5webengine/Config.in
index a05205c7ad..56b6da8b0e 100644
--- a/package/qt5/qt5webengine/Config.in
+++ b/package/qt5/qt5webengine/Config.in
@@ -10,10 +10,9 @@ 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/ gcc >= 4.8, threads, wchar"
+comment "qt5webengine needs udev /dev management and a glibc toolchain w/ threads, wchar"
 	depends on BR2_PACKAGE_QT5WEBENGINE_ARCH_SUPPORTS
 	depends on !BR2_PACKAGE_HAS_UDEV || !BR2_TOOLCHAIN_USES_GLIBC || \
-		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
 		!BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR
 
 comment "qt5webengine needs an OpenGL and EGL-capable backend"
@@ -24,7 +23,6 @@ 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_4_8 # qt5base-icu
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2, libvpx, qt5base-dbus
 	depends on BR2_USE_WCHAR # libglib2
 	depends on BR2_PACKAGE_QT5_GL_AVAILABLE # qt5declarative, qt5base-eglfs
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 5/6] package/qt5webkit: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
                   ` (2 preceding siblings ...)
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 4/6] package/qt5webengine: " Peter Seiderer
@ 2020-04-18 22:59 ` Peter Seiderer
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 6/6] package/qt-webkit-kiosk: " Peter Seiderer
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/qt5/qt5webkit/Config.in | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/package/qt5/qt5webkit/Config.in b/package/qt5/qt5webkit/Config.in
index e071dff208..5adcddfb5d 100644
--- a/package/qt5/qt5webkit/Config.in
+++ b/package/qt5/qt5webkit/Config.in
@@ -3,7 +3,6 @@ config BR2_PACKAGE_QT5WEBKIT
 	depends on !BR2_STATIC_LIBS
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	depends on BR2_INSTALL_LIBSTDCPP # leveldb
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # icu
 	depends on BR2_TOOLCHAIN_HAS_THREADS # leveldb
 	depends on !BR2_BINFMT_FLAT # icu
 	# assumes a FPU is available on MIPS
@@ -31,8 +30,8 @@ config BR2_PACKAGE_QT5WEBKIT
 
 	  http://doc.qt.io/archives/qt-5.5/qtwebkit-index.html
 
-comment "qt5webkit needs a toolchain w/ dynamic library, gcc >= 4.8"
-	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+comment "qt5webkit needs a toolchain w/ dynamic library"
+	depends on BR2_STATIC_LIBS
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	depends on !BR2_BINFMT_FLAT
 	depends on !BR2_MIPS_SOFT_FLOAT
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 6/6] package/qt-webkit-kiosk: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
                   ` (3 preceding siblings ...)
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 5/6] package/qt5webkit: " Peter Seiderer
@ 2020-04-18 22:59 ` Peter Seiderer
  2020-04-19 13:00 ` [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Thomas Petazzoni
  2020-05-06  5:48 ` Peter Korsgaard
  6 siblings, 0 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-18 22:59 UTC (permalink / raw)
  To: buildroot

As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/qt-webkit-kiosk/Config.in | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/package/qt-webkit-kiosk/Config.in b/package/qt-webkit-kiosk/Config.in
index 46688a227b..b620154ed1 100644
--- a/package/qt-webkit-kiosk/Config.in
+++ b/package/qt-webkit-kiosk/Config.in
@@ -3,7 +3,6 @@ config BR2_PACKAGE_QT_WEBKIT_KIOSK
 	depends on BR2_PACKAGE_QT5
 	depends on !BR2_STATIC_LIBS # qt5webkit
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE # qt5webkit
-	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 # qt5webkit -> icu
 	depends on !BR2_BINFMT_FLAT # qt5webkit -> icu
 	depends on !BR2_MIPS_SOFT_FLOAT # qt5webkit
 	select BR2_PACKAGE_OPENSSL
@@ -25,8 +24,8 @@ config BR2_PACKAGE_QT_WEBKIT_KIOSK_SOUNDS
 
 endif
 
-comment "qt-webkit-kiosk needs a toolchain w/ dynamic library, gcc >= 4.8"
-	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
+comment "qt-webkit-kiosk needs a toolchain w/ dynamic library"
+	depends on BR2_STATIC_LIBS
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	depends on !BR2_BINFMT_FLAT
 	depends on !BR2_MIPS_SOFT_FLOAT
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
                   ` (4 preceding siblings ...)
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 6/6] package/qt-webkit-kiosk: " Peter Seiderer
@ 2020-04-19 13:00 ` Thomas Petazzoni
  2020-05-06  5:48 ` Peter Korsgaard
  6 siblings, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2020-04-19 13:00 UTC (permalink / raw)
  To: buildroot

On Sun, 19 Apr 2020 00:59:01 +0200
Peter Seiderer <ps.report@gmx.net> wrote:

> As a side effect of avoiding gcc bug 57694 ([1]) through commit
> 'GCC: revoke constexpr before 5.0' ([2]) for qtbase the minimum
> gcc requirement was lifted to 5.0 ([3], [4]), so bump the
> buildroot determined one too.
> 
> Adjust the two packages (pinentry, wireshark) which are brave enough
> to do a 'select BR2_PACKAGE_QT5' accordingly.

Huhu, "brave enough" :-)

> Fixes:
>   http://autobuild.buildroot.net/results/74f5249d1ee14c1269685af53a65055da7dc2ee6
> 
>   ../../include/QtCore/../../src/corelib/thread/qbasicatomic.h:61:4: error: #error "Qt requires C++11 support"
>    #  error "Qt requires C++11 support"
> 
> [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57694
> [2] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=c61cedcc5475a77fb94ed12788f61039835f8079
> [3] https://codereview.qt-project.org/c/qt/qtbase/+/285582
> [4] https://bugreports.qt.io/browse/QTBUG-81761
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
>  package/pinentry/Config.in  | 6 +++---
>  package/qt5/Config.in       | 6 +++---
>  package/wireshark/Config.in | 6 +++---
>  3 files changed, 9 insertions(+), 9 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0
  2020-04-18 22:59 ` [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0 Peter Seiderer
@ 2020-04-19 13:03   ` Thomas Petazzoni
  2020-04-20 18:35     ` Peter Seiderer
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2020-04-19 13:03 UTC (permalink / raw)
  To: buildroot

On Sun, 19 Apr 2020 00:59:02 +0200
Peter Seiderer <ps.report@gmx.net> wrote:

> As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
>  package/qt5/qt5base/Config.in | 9 +--------
>  1 file changed, 1 insertion(+), 8 deletions(-)

I have not applied this one, and neither the remaining patches in the
series. Indeed, we normally want to propagate "depends on" to all
reverse dependencies, even if in practice it is already not possible to
select a given option due to it being hidden by a "depends on" at a
higher level.

We believe that always duplicating improves consistency and helps
avoiding mistakes. I agree that we are not 100% consistent on this
throughout the tree, but the direction we want to go to is to replicate
these "depends no", not get rid of them.

I agree that this is a very 'opinionated' decision and I am myself a
bit split between the two possibilities we have. But the overall
consensus seems to be that we should have that duplication.

So, I've marked PATCH 2/6 to 6/6 as Rejected.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0
  2020-04-19 13:03   ` Thomas Petazzoni
@ 2020-04-20 18:35     ` Peter Seiderer
  0 siblings, 0 replies; 12+ messages in thread
From: Peter Seiderer @ 2020-04-20 18:35 UTC (permalink / raw)
  To: buildroot

Hello Thomas,

On Sun, 19 Apr 2020 15:03:05 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

> On Sun, 19 Apr 2020 00:59:02 +0200
> Peter Seiderer <ps.report@gmx.net> wrote:
>
> > As qt5 requires as minimum gcc 5.0 drop all legacy requierements below.
> >
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > ---
> >  package/qt5/qt5base/Config.in | 9 +--------
> >  1 file changed, 1 insertion(+), 8 deletions(-)
>
> I have not applied this one, and neither the remaining patches in the
> series. Indeed, we normally want to propagate "depends on" to all
> reverse dependencies, even if in practice it is already not possible to
> select a given option due to it being hidden by a "depends on" at a
> higher level.
>
> We believe that always duplicating improves consistency and helps
> avoiding mistakes. I agree that we are not 100% consistent on this
> throughout the tree, but the direction we want to go to is to replicate
> these "depends no", not get rid of them.
>
> I agree that this is a very 'opinionated' decision and I am myself a
> bit split between the two possibilities we have. But the overall
> consensus seems to be that we should have that duplication.
>
> So, I've marked PATCH 2/6 to 6/6 as Rejected.

Fine with me ;-), and thanks for the detailed explanation...

Regards,
Peter

>
> Thanks!
>
> Thomas

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0
  2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
                   ` (5 preceding siblings ...)
  2020-04-19 13:00 ` [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Thomas Petazzoni
@ 2020-05-06  5:48 ` Peter Korsgaard
  2020-05-06  6:44   ` Peter Seiderer
  6 siblings, 1 reply; 12+ messages in thread
From: Peter Korsgaard @ 2020-05-06  5:48 UTC (permalink / raw)
  To: buildroot

>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:

 > As a side effect of avoiding gcc bug 57694 ([1]) through commit
 > 'GCC: revoke constexpr before 5.0' ([2]) for qtbase the minimum
 > gcc requirement was lifted to 5.0 ([3], [4]), so bump the
 > buildroot determined one too.

 > Adjust the two packages (pinentry, wireshark) which are brave enough
 > to do a 'select BR2_PACKAGE_QT5' accordingly.

 > Fixes:
 >   http://autobuild.buildroot.net/results/74f5249d1ee14c1269685af53a65055da7dc2ee6

 >   ../../include/QtCore/../../src/corelib/thread/qbasicatomic.h:61:4: error: #error "Qt requires C++11 support"
 >    #  error "Qt requires C++11 support"

 > [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57694
 > [2] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=c61cedcc5475a77fb94ed12788f61039835f8079
 > [3] https://codereview.qt-project.org/c/qt/qtbase/+/285582
 > [4] https://bugreports.qt.io/browse/QTBUG-81761

 > Signed-off-by: Peter Seiderer <ps.report@gmx.net>

Does this also apply to the qt5 version we have in 2020.02.x
(E.G. 5.12.8)?

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0
  2020-05-06  5:48 ` Peter Korsgaard
@ 2020-05-06  6:44   ` Peter Seiderer
  2020-05-06  7:37     ` Peter Korsgaard
  0 siblings, 1 reply; 12+ messages in thread
From: Peter Seiderer @ 2020-05-06  6:44 UTC (permalink / raw)
  To: buildroot

Hello Peter,

On Wed, 06 May 2020 07:48:34 +0200, Peter Korsgaard <peter@korsgaard.com> wrote:

> >>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:
>
>  > As a side effect of avoiding gcc bug 57694 ([1]) through commit
>  > 'GCC: revoke constexpr before 5.0' ([2]) for qtbase the minimum
>  > gcc requirement was lifted to 5.0 ([3], [4]), so bump the
>  > buildroot determined one too.
>
>  > Adjust the two packages (pinentry, wireshark) which are brave enough
>  > to do a 'select BR2_PACKAGE_QT5' accordingly.
>
>  > Fixes:
>  >   http://autobuild.buildroot.net/results/74f5249d1ee14c1269685af53a65055da7dc2ee6
>
>  >   ../../include/QtCore/../../src/corelib/thread/qbasicatomic.h:61:4: error: #error "Qt requires C++11 support"
>  >    #  error "Qt requires C++11 support"
>
>  > [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57694
>  > [2] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=c61cedcc5475a77fb94ed12788f61039835f8079
>  > [3] https://codereview.qt-project.org/c/qt/qtbase/+/285582
>  > [4] https://bugreports.qt.io/browse/QTBUG-81761
>
>  > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
>
> Does this also apply to the qt5 version we have in 2020.02.x
> (E.G. 5.12.8)?
>

Qt-5.12.8 should not be affected as commit [2] is not part of this release...

Regards,
Peter

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0
  2020-05-06  6:44   ` Peter Seiderer
@ 2020-05-06  7:37     ` Peter Korsgaard
  0 siblings, 0 replies; 12+ messages in thread
From: Peter Korsgaard @ 2020-05-06  7:37 UTC (permalink / raw)
  To: buildroot

>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:

 > Hello Peter,
 > On Wed, 06 May 2020 07:48:34 +0200, Peter Korsgaard <peter@korsgaard.com> wrote:

 >> 
 >> Does this also apply to the qt5 version we have in 2020.02.x
 >> (E.G. 5.12.8)?
 >> 

 > Qt-5.12.8 should not be affected as commit [2] is not part of this release...

Ok, thanks for the confirmation!

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2020-05-06  7:37 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-18 22:59 [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Peter Seiderer
2020-04-18 22:59 ` [Buildroot] [PATCH v1 2/6] package/qt5base: drop all legacy gcc requierements below 5.0 Peter Seiderer
2020-04-19 13:03   ` Thomas Petazzoni
2020-04-20 18:35     ` Peter Seiderer
2020-04-18 22:59 ` [Buildroot] [PATCH v1 3/6] package/qt5tools: " Peter Seiderer
2020-04-18 22:59 ` [Buildroot] [PATCH v1 4/6] package/qt5webengine: " Peter Seiderer
2020-04-18 22:59 ` [Buildroot] [PATCH v1 5/6] package/qt5webkit: " Peter Seiderer
2020-04-18 22:59 ` [Buildroot] [PATCH v1 6/6] package/qt-webkit-kiosk: " Peter Seiderer
2020-04-19 13:00 ` [Buildroot] [PATCH v1 1/6] package/qt5: bump minimum gcc requirement to 5.0 Thomas Petazzoni
2020-05-06  5:48 ` Peter Korsgaard
2020-05-06  6:44   ` Peter Seiderer
2020-05-06  7:37     ` Peter Korsgaard

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.