diff mbox

[2/7] eigen: add C++ toolchain dependency

Message ID 1384203613-17871-3-git-send-email-s.martin49@gmail.com
State Accepted
Commit 76f02e371a3b9d73d62bd6a74a85437b9dec7f9a
Headers show

Commit Message

Samuel Martin Nov. 11, 2013, 9 p.m. UTC
As said in its help text, eigen heavily uses C++ features.
So, explicit this dependency in the Config.in.

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
---
 package/eigen/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Arnout Vandecappelle Nov. 12, 2013, 7:15 a.m. UTC | #1
On 11/11/13 22:00, Samuel Martin wrote:
> As said in its help text, eigen heavily uses C++ features.
> So, explicit this dependency in the Config.in.
>
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

> ---
>   package/eigen/Config.in | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/package/eigen/Config.in b/package/eigen/Config.in
> index c7fcde2..e94f9a3 100644
> --- a/package/eigen/Config.in
> +++ b/package/eigen/Config.in
> @@ -1,5 +1,6 @@
>   config BR2_PACKAGE_EIGEN
>   	bool "eigen"
> +	depends on BR2_INSTALL_LIBSTDCPP
>   	help
>   	  Eigen is a C++ template library for linear algebra: vectors,
>   	  matrices, and related algorithms. It is versatile, fast, elegant
> @@ -11,3 +12,6 @@ config BR2_PACKAGE_EIGEN
>   	  template library defined in the headers.
>
>   	  http://eigen.tuxfamily.org/
> +
> +comment "eigen needs a toolchain w/ C++"
> +	depends on !BR2_INSTALL_LIBSTDCPP
>
diff mbox

Patch

diff --git a/package/eigen/Config.in b/package/eigen/Config.in
index c7fcde2..e94f9a3 100644
--- a/package/eigen/Config.in
+++ b/package/eigen/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_EIGEN
 	bool "eigen"
+	depends on BR2_INSTALL_LIBSTDCPP
 	help
 	  Eigen is a C++ template library for linear algebra: vectors,
 	  matrices, and related algorithms. It is versatile, fast, elegant
@@ -11,3 +12,6 @@  config BR2_PACKAGE_EIGEN
 	  template library defined in the headers.
 
 	  http://eigen.tuxfamily.org/
+
+comment "eigen needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP