diff mbox

[3/3] python-treq: add missing BR2_INSTALL_LIBSTDCPP dependency

Message ID 1463521199-13662-3-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Headers show

Commit Message

Thomas Petazzoni May 17, 2016, 9:39 p.m. UTC
python-treq selects python-pyopenssl, which depends on C++ support, but
this dependency was not propagated to python-treq, causing the following
kconfig warning:

warning: (BR2_PACKAGE_PYTHON_TREQ) selects BR2_PACKAGE_PYTHON_PYOPENSSL which has unmet direct dependencies ((BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3) && BR2_INSTALL_LIBSTDCPP)

This commit fixes that by adding the appropriate dependency.

Cc: Yegor Yefremov <yegorslists@googlemail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/python-treq/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Yegor Yefremov May 24, 2016, 7:07 a.m. UTC | #1
On Tue, May 17, 2016 at 11:39 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> python-treq selects python-pyopenssl, which depends on C++ support, but
> this dependency was not propagated to python-treq, causing the following
> kconfig warning:
>
> warning: (BR2_PACKAGE_PYTHON_TREQ) selects BR2_PACKAGE_PYTHON_PYOPENSSL which has unmet direct dependencies ((BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3) && BR2_INSTALL_LIBSTDCPP)
>
> This commit fixes that by adding the appropriate dependency.
>
> Cc: Yegor Yefremov <yegorslists@googlemail.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>

> ---
>  package/python-treq/Config.in | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/package/python-treq/Config.in b/package/python-treq/Config.in
> index a8590a9..d560ba3 100644
> --- a/package/python-treq/Config.in
> +++ b/package/python-treq/Config.in
> @@ -5,6 +5,7 @@ config BR2_PACKAGE_PYTHON_TREQ
>         select BR2_PACKAGE_PYTHON_SIX # runtime
>         select BR2_PACKAGE_PYTHON_TWISTED # runtime
>         select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
> +       depends on BR2_INSTALL_LIBSTDCPP # python-pyopenssl
>         help
>           treq is an HTTP library inspired by requests but written on
>           top of Twisted's Agents.  It provides a simple, higher level
> --
> 2.7.4
>
Thomas Petazzoni May 24, 2016, 3:25 p.m. UTC | #2
Hello,

On Tue, 17 May 2016 23:39:59 +0200, Thomas Petazzoni wrote:

> diff --git a/package/python-treq/Config.in b/package/python-treq/Config.in
> index a8590a9..d560ba3 100644
> --- a/package/python-treq/Config.in
> +++ b/package/python-treq/Config.in
> @@ -5,6 +5,7 @@ config BR2_PACKAGE_PYTHON_TREQ
>  	select BR2_PACKAGE_PYTHON_SIX # runtime
>  	select BR2_PACKAGE_PYTHON_TWISTED # runtime
>  	select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
> +	depends on BR2_INSTALL_LIBSTDCPP # python-pyopenssl
>  	help
>  	  treq is an HTTP library inspired by requests but written on
>  	  top of Twisted's Agents.  It provides a simple, higher level

Same here, I've added the Config.in comment for the dependency, which
was missing in my patch, and applied to master.

Thomas
diff mbox

Patch

diff --git a/package/python-treq/Config.in b/package/python-treq/Config.in
index a8590a9..d560ba3 100644
--- a/package/python-treq/Config.in
+++ b/package/python-treq/Config.in
@@ -5,6 +5,7 @@  config BR2_PACKAGE_PYTHON_TREQ
 	select BR2_PACKAGE_PYTHON_SIX # runtime
 	select BR2_PACKAGE_PYTHON_TWISTED # runtime
 	select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+	depends on BR2_INSTALL_LIBSTDCPP # python-pyopenssl
 	help
 	  treq is an HTTP library inspired by requests but written on
 	  top of Twisted's Agents.  It provides a simple, higher level