diff mbox

[v2,1/2] qt: Script Module needs a toolchain with nptl

Message ID 1415962341-9642-1-git-send-email-Vincent.Riera@imgtec.com
State Superseded
Headers show

Commit Message

Vicente Olivert Riera Nov. 14, 2014, 10:52 a.m. UTC
Qt Script Module uses pthread_getattr_np() which is implemented in
toolchains with nptl. Otherwise the compilation will fail showing this
error message:

../3rdparty/javascriptcore/JavaScriptCore/runtime/Collector.cpp: In
function ‘void* QTJSC::currentThreadStackBase()’:
../3rdparty/javascriptcore/JavaScriptCore/runtime/Collector.cpp:692:42:
error: ‘pthread_getattr_np’ was not declared in this scope

Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
---
Changes v1 -> v2

Depend on BR2_TOOLCHAIN_HAS_THREADS_NPTL instead of BR2_PTHREADS_OLD

 package/qt/Config.in |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

Comments

Arnout Vandecappelle Nov. 14, 2014, 10:50 p.m. UTC | #1
On 14/11/14 11:52, Vicente Olivert Riera wrote:
> Qt Script Module uses pthread_getattr_np() which is implemented in
> toolchains with nptl. Otherwise the compilation will fail showing this
> error message:
> 
> ../3rdparty/javascriptcore/JavaScriptCore/runtime/Collector.cpp: In
> function ‘void* QTJSC::currentThreadStackBase()’:
> ../3rdparty/javascriptcore/JavaScriptCore/runtime/Collector.cpp:692:42:
> error: ‘pthread_getattr_np’ was not declared in this scope
> 
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
> Changes v1 -> v2
> 
> Depend on BR2_TOOLCHAIN_HAS_THREADS_NPTL instead of BR2_PTHREADS_OLD
> 
>  package/qt/Config.in |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/package/qt/Config.in b/package/qt/Config.in
> index da159b4..f4cb6b6 100644
> --- a/package/qt/Config.in
> +++ b/package/qt/Config.in
> @@ -402,11 +402,16 @@ config BR2_PACKAGE_QT_OPENSSL
>  config BR2_PACKAGE_QT_SCRIPT
>  	bool "Script Module"
>  	depends on BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
> +	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # needs pthread_getattr_np()
>  	default y
>  	help
>  	  Build the Qt Script module.
>  	  if unsure, say y.
>  
> +comment "Script Module needs a toolchain with nptl"

 NPTL should be in capitals.

 Regards,
 Arnout

> +	depends on !BR2_TOOLCHAIN_HAS_THREADS_NPTL
> +	depends on BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
> +
>  config BR2_PACKAGE_QT_SCRIPTTOOLS
>  	bool "Script Tools Module"
>  	depends on BR2_PACKAGE_QT_SCRIPT
>
diff mbox

Patch

diff --git a/package/qt/Config.in b/package/qt/Config.in
index da159b4..f4cb6b6 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -402,11 +402,16 @@  config BR2_PACKAGE_QT_OPENSSL
 config BR2_PACKAGE_QT_SCRIPT
 	bool "Script Module"
 	depends on BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
+	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # needs pthread_getattr_np()
 	default y
 	help
 	  Build the Qt Script module.
 	  if unsure, say y.
 
+comment "Script Module needs a toolchain with nptl"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS_NPTL
+	depends on BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
+
 config BR2_PACKAGE_QT_SCRIPTTOOLS
 	bool "Script Tools Module"
 	depends on BR2_PACKAGE_QT_SCRIPT