diff mbox series

[v1,4/4] package/wireshark: add missing qt5 dependencies

Message ID 20200310081902.3488-4-ps.report@gmx.net
State Accepted
Headers show
Series [v1,1/4] package/qt5: enable for nios2 | expand

Commit Message

Peter Seiderer March 10, 2020, 8:19 a.m. UTC
- BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
  - !BR2_arc

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

Comments

Thomas Petazzoni March 15, 2020, 10:39 p.m. UTC | #1
On Tue, 10 Mar 2020 09:19:02 +0100
Peter Seiderer <ps.report@gmx.net> wrote:

> -comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL"
> +comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
>  	depends on !BR2_PACKAGE_HAS_UDEV || !BR2_INSTALL_LIBSTDCPP || \
> -		!BR2_TOOLCHAIN_HAS_THREADS_NPTL
> +		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
> +		!BR2_arc

Same problem as for pinentry in how the arch dependency is taken into
account.

Fixed to:

comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
        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

And applied. Thanks!

Thomas
Peter Korsgaard March 27, 2020, 6:42 a.m. UTC | #2
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

 > On Tue, 10 Mar 2020 09:19:02 +0100
 > Peter Seiderer <ps.report@gmx.net> wrote:

 >> -comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL"
 >> +comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
 >> depends on !BR2_PACKAGE_HAS_UDEV || !BR2_INSTALL_LIBSTDCPP || \
 >> -		!BR2_TOOLCHAIN_HAS_THREADS_NPTL
 >> +		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
 >> +		!BR2_arc

 > Same problem as for pinentry in how the arch dependency is taken into
 > account.

 > Fixed to:

 > comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
 >         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

 > And applied. Thanks!

Committed to 2020.02.x, thanks.
diff mbox series

Patch

diff --git a/package/wireshark/Config.in b/package/wireshark/Config.in
index 382e429a9c..8e3bc7aad5 100644
--- a/package/wireshark/Config.in
+++ b/package/wireshark/Config.in
@@ -21,6 +21,8 @@  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_arc # qt5
 	depends on BR2_PACKAGE_HAS_UDEV # qt5 input
 	select BR2_PACKAGE_QT5
 	select BR2_PACKAGE_QT5BASE
@@ -34,9 +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"
+comment "Qt5 GUI needs a udev /dev management and a toolchain w/ C++, NPTL, gcc >= 4.8"
 	depends on !BR2_PACKAGE_HAS_UDEV || !BR2_INSTALL_LIBSTDCPP || \
-		!BR2_TOOLCHAIN_HAS_THREADS_NPTL
+		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
+		!BR2_arc
 
 endif # BR2_PACKAGE_WIRESHARK