diff mbox series

nghttp2: Always enable python bindings

Message ID 20230128233021.2230473-1-raj.khem@gmail.com
State New
Headers show
Series nghttp2: Always enable python bindings | expand

Commit Message

Khem Raj Jan. 28, 2023, 11:30 p.m. UTC
This makes the build host agnostic, otherwise on some hosts e.g. (archlinux)
it ends up using cython compiler from build host, mixing python3-native
and cython from host is not a tested combination and can result in
obscure errors. Lets make it use cython from OE native packages and also
enable the bindings explicitly.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-support/nghttp2/nghttp2_1.51.0.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Khem Raj Jan. 29, 2023, 5 a.m. UTC | #1
I have sent a v2, please disregard this one.

On Sat, Jan 28, 2023 at 3:30 PM Khem Raj <raj.khem@gmail.com> wrote:
>
> This makes the build host agnostic, otherwise on some hosts e.g. (archlinux)
> it ends up using cython compiler from build host, mixing python3-native
> and cython from host is not a tested combination and can result in
> obscure errors. Lets make it use cython from OE native packages and also
> enable the bindings explicitly.
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  meta/recipes-support/nghttp2/nghttp2_1.51.0.bb | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb b/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
> index fc1a592200..4402b80d1a 100644
> --- a/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
> +++ b/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
> @@ -4,6 +4,8 @@ SECTION = "libs"
>  LICENSE = "MIT"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=764abdf30b2eadd37ce47dcbce0ea1ec"
>
> +DEPENDS += "python3-cython-native"
> +
>  SRC_URI = "\
>      ${GITHUB_BASE_URI}/download/v${PV}/nghttp2-${PV}.tar.xz \
>      file://0001-fetch-ocsp-response-use-python3.patch \
> @@ -15,7 +17,7 @@ PACKAGECONFIG[manpages] = ""
>
>  # examples are never installed, and don't need to be built in the
>  # first place
> -EXTRA_OECMAKE = "-DENABLE_EXAMPLES=OFF -DENABLE_APP=OFF -DENABLE_HPACK_TOOLS=OFF"
> +EXTRA_OECMAKE = "-DENABLE_EXAMPLES=OFF -DENABLE_APP=OFF -DENABLE_HPACK_TOOLS=OFF -DENABLE_PYTHON_BINDINGS=ON"
>
>  PACKAGES =+ "lib${BPN} ${PN}-client ${PN}-proxy ${PN}-server"
>
> --
> 2.39.1
>
diff mbox series

Patch

diff --git a/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb b/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
index fc1a592200..4402b80d1a 100644
--- a/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
+++ b/meta/recipes-support/nghttp2/nghttp2_1.51.0.bb
@@ -4,6 +4,8 @@  SECTION = "libs"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://COPYING;md5=764abdf30b2eadd37ce47dcbce0ea1ec"
 
+DEPENDS += "python3-cython-native"
+
 SRC_URI = "\
     ${GITHUB_BASE_URI}/download/v${PV}/nghttp2-${PV}.tar.xz \
     file://0001-fetch-ocsp-response-use-python3.patch \
@@ -15,7 +17,7 @@  PACKAGECONFIG[manpages] = ""
 
 # examples are never installed, and don't need to be built in the
 # first place
-EXTRA_OECMAKE = "-DENABLE_EXAMPLES=OFF -DENABLE_APP=OFF -DENABLE_HPACK_TOOLS=OFF"
+EXTRA_OECMAKE = "-DENABLE_EXAMPLES=OFF -DENABLE_APP=OFF -DENABLE_HPACK_TOOLS=OFF -DENABLE_PYTHON_BINDINGS=ON"
 
 PACKAGES =+ "lib${BPN} ${PN}-client ${PN}-proxy ${PN}-server"