diff mbox series

[v8,05/26] package/python-pyyaml: add cython for host-python-pyyaml

Message ID 20240213211512.3918044-5-james.hilliard1@gmail.com
State Accepted
Headers show
Series [v8,01/26] package/python-poetry-core: new host package | expand

Commit Message

James Hilliard Feb. 13, 2024, 9:14 p.m. UTC
When migrating to setuptools with pep517 support we need to add
host-python-cython which is a pep517 build dependency for pyyaml.

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
---
 package/python-pyyaml/python-pyyaml.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Arnout Vandecappelle March 27, 2024, 10 p.m. UTC | #1
On 13/02/2024 22:14, James Hilliard wrote:
> When migrating to setuptools with pep517 support we need to add
> host-python-cython which is a pep517 build dependency for pyyaml.
> 
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
> ---
>   package/python-pyyaml/python-pyyaml.mk | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk
> index 846c57df4f..4ab91ca5f6 100644
> --- a/package/python-pyyaml/python-pyyaml.mk
> +++ b/package/python-pyyaml/python-pyyaml.mk
> @@ -14,7 +14,8 @@ PYTHON_PYYAML_CPE_ID_VENDOR = pyyaml
>   PYTHON_PYYAML_CPE_ID_PRODUCT = pyyaml
>   PYTHON_PYYAML_DEPENDENCIES = host-python-cython libyaml
>   PYTHON_PYYAML_ENV = PYYAML_FORCE_CYTHON=1
> -HOST_PYTHON_PYYAML_DEPENDENCIES = host-libyaml
> +HOST_PYTHON_PYYAML_DEPENDENCIES = host-python-cython host-libyaml

  Again, I ordered the dependencies alphabetically.

  Applied to master, thanks.

  Regards,
  Arnout

> +HOST_PYTHON_PYYAML_ENV = PYYAML_FORCE_CYTHON=1
>   
>   $(eval $(python-package))
>   $(eval $(host-python-package))
diff mbox series

Patch

diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk
index 846c57df4f..4ab91ca5f6 100644
--- a/package/python-pyyaml/python-pyyaml.mk
+++ b/package/python-pyyaml/python-pyyaml.mk
@@ -14,7 +14,8 @@  PYTHON_PYYAML_CPE_ID_VENDOR = pyyaml
 PYTHON_PYYAML_CPE_ID_PRODUCT = pyyaml
 PYTHON_PYYAML_DEPENDENCIES = host-python-cython libyaml
 PYTHON_PYYAML_ENV = PYYAML_FORCE_CYTHON=1
-HOST_PYTHON_PYYAML_DEPENDENCIES = host-libyaml
+HOST_PYTHON_PYYAML_DEPENDENCIES = host-python-cython host-libyaml
+HOST_PYTHON_PYYAML_ENV = PYYAML_FORCE_CYTHON=1
 
 $(eval $(python-package))
 $(eval $(host-python-package))