[Buildroot] [PATCH] package/qt: enable static build for BR2_PACKAGE_QT_MOUSE_TSLIB

[Buildroot] [PATCH] package/qt: enable static build for BR2_PACKAGE_QT_MOUSE_TSLIB

[Martin Kepplinger]

The dependency on !BR2_STATIC_LIBS was here because of tslib not supporing
static builds. But since

   commit 05fadfea1 ("package/tslib: enable statically linked building")

it does and we can remove this dependecy.

Signed-off-by: Martin Kepplinger <martin.kepplinger@ginzinger.com>
---
 package/qt/Config.mouse.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/package/qt/Config.mouse.in b/package/qt/Config.mouse.in
index 07573ebb8..eb9d2c0c2 100644
--- a/package/qt/Config.mouse.in
+++ b/package/qt/Config.mouse.in
@@ -11,7 +11,6 @@ config BR2_PACKAGE_QT_MOUSE_LINUXINPUT
 
 config BR2_PACKAGE_QT_MOUSE_TSLIB
 	bool "tslib"
-	depends on !BR2_STATIC_LIBS # tslib
 	select BR2_PACKAGE_TSLIB
 
 comment "tslib support needs a toolchain w/ dynamic library"
-- 
2.11.0

[Buildroot] [PATCH] package/qt: enable static build for BR2_PACKAGE_QT_MOUSE_TSLIB

[Baruch Siach]

Hi Martin,

On Thu, Jun 08, 2017 at 11:02:59AM +0200, Martin Kepplinger wrote:
> The dependency on !BR2_STATIC_LIBS was here because of tslib not supporing
> static builds. But since
> 
>    commit 05fadfea1 ("package/tslib: enable statically linked building")
> 
> it does and we can remove this dependecy.
> 
> Signed-off-by: Martin Kepplinger <martin.kepplinger@ginzinger.com>
> ---
>  package/qt/Config.mouse.in | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/package/qt/Config.mouse.in b/package/qt/Config.mouse.in
> index 07573ebb8..eb9d2c0c2 100644
> --- a/package/qt/Config.mouse.in
> +++ b/package/qt/Config.mouse.in
> @@ -11,7 +11,6 @@ config BR2_PACKAGE_QT_MOUSE_LINUXINPUT
>  
>  config BR2_PACKAGE_QT_MOUSE_TSLIB
>  	bool "tslib"
> -	depends on !BR2_STATIC_LIBS # tslib
>  	select BR2_PACKAGE_TSLIB
>  
>  comment "tslib support needs a toolchain w/ dynamic library"

This comment is not correct anymore.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -

[Buildroot] [PATCH v2] package/qt: enable static build for BR2_PACKAGE_QT_MOUSE_TSLIB

[Martin Kepplinger]

The dependency on !BR2_STATIC_LIBS was here because of tslib not supporing
static builds. But since

   commit 05fadfea1 ("package/tslib: enable statically linked building")

it does and we can remove this dependecy.

Signed-off-by: Martin Kepplinger <martin.kepplinger@ginzinger.com>
---

v2:
Thanks Baruch Siach for reviewing. I missed removing the dependecy on
BR2_STATIC_LIBS


 package/qt/Config.mouse.in | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/package/qt/Config.mouse.in b/package/qt/Config.mouse.in
index 07573ebb8..1062facb6 100644
--- a/package/qt/Config.mouse.in
+++ b/package/qt/Config.mouse.in
@@ -11,12 +11,8 @@ config BR2_PACKAGE_QT_MOUSE_LINUXINPUT
 
 config BR2_PACKAGE_QT_MOUSE_TSLIB
 	bool "tslib"
-	depends on !BR2_STATIC_LIBS # tslib
 	select BR2_PACKAGE_TSLIB
 
-comment "tslib support needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
-
 config BR2_PACKAGE_QT_MOUSE_QVFB
 	bool "qvfb"
 
-- 
2.11.0

[Buildroot] [PATCH v2] package/qt: enable static build for BR2_PACKAGE_QT_MOUSE_TSLIB

[Thomas Petazzoni]

Hello,

On Thu, 8 Jun 2017 12:03:12 +0200, Martin Kepplinger wrote:
> The dependency on !BR2_STATIC_LIBS was here because of tslib not supporing
> static builds. But since
> 
>    commit 05fadfea1 ("package/tslib: enable statically linked building")
> 
> it does and we can remove this dependecy.
> 
> Signed-off-by: Martin Kepplinger <martin.kepplinger@ginzinger.com>
> ---

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com