diff mbox

[3/6] qt: remove broken iBase driver

Message ID 20170307235054.25149-3-arnout@mind.be
State Accepted
Headers show

Commit Message

Arnout Vandecappelle March 7, 2017, 11:50 p.m. UTC
This option has been marked as broken since 2010. No need for legacy
handling.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/qt/Config.sql.in | 7 -------
 1 file changed, 7 deletions(-)

Comments

Peter Seiderer March 8, 2017, 7:19 a.m. UTC | #1
Hello Arnout,

> Gesendet: Mittwoch, 08. März 2017 um 00:50 Uhr
> Von: "Arnout Vandecappelle (Essensium/Mind)" <arnout@mind.be>
> An: buildroot@buildroot.org
> Betreff: [Buildroot] [PATCH 3/6] qt: remove broken iBase driver
>
> This option has been marked as broken since 2010. No need for legacy
> handling.
> 
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/qt/Config.sql.in | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/package/qt/Config.sql.in b/package/qt/Config.sql.in
> index 452edd5133..5bd63d27a0 100644
> --- a/package/qt/Config.sql.in
> +++ b/package/qt/Config.sql.in
> @@ -14,13 +14,6 @@ config BR2_PACKAGE_QT_MYSQL
>  	  Build MySQL driver
>  	  If unsure, say n.
>  
> -config BR2_PACKAGE_QT_IBASE
> -	bool "iBase Driver"
> -	depends on BROKEN # libfbclient not in BR
> -	help
> -	  Build iBase driver
> -	  If unsure, say n.
> -
>  config BR2_PACKAGE_QT_ODBC
>  	bool "ODBC Driver"
>  	select BR2_PACKAGE_UNIXODBC
> -- 
> 2.11.0
> 

Missing some code deletion from package/qt/qt.mk file:

ifeq ($(BR2_PACKAGE_QT_IBASE),y)
QT_CONFIGURE_OPTS += -qt-sql-ibase
endif

Regards,
Peter

> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
Thomas Petazzoni March 9, 2017, 9:30 p.m. UTC | #2
Hello,

On Wed, 8 Mar 2017 08:19:19 +0100, Peter Seiderer wrote:

> Missing some code deletion from package/qt/qt.mk file:
> 
> ifeq ($(BR2_PACKAGE_QT_IBASE),y)
> QT_CONFIGURE_OPTS += -qt-sql-ibase
> endif

Well spotted, I've tweaked Arnout's patch when applying to remove this
part of the code. Thanks!

Thomas
diff mbox

Patch

diff --git a/package/qt/Config.sql.in b/package/qt/Config.sql.in
index 452edd5133..5bd63d27a0 100644
--- a/package/qt/Config.sql.in
+++ b/package/qt/Config.sql.in
@@ -14,13 +14,6 @@  config BR2_PACKAGE_QT_MYSQL
 	  Build MySQL driver
 	  If unsure, say n.
 
-config BR2_PACKAGE_QT_IBASE
-	bool "iBase Driver"
-	depends on BROKEN # libfbclient not in BR
-	help
-	  Build iBase driver
-	  If unsure, say n.
-
 config BR2_PACKAGE_QT_ODBC
 	bool "ODBC Driver"
 	select BR2_PACKAGE_UNIXODBC