On 26 Sep 2024, at 9:24, Roi Dayan via dev wrote:

> Allow passing different DEB_BUILD_OPTIONS to make debian-deb.
>
> Signed-off-by: Roi Dayan <[email protected]>

Hi Roy,

Not sure if you noticed, but the robot failed to build you change, i.e. the 
github CI failed to build debian.

https://github.com/ovsrobot/ovs/actions/runs/11047719338

I’ll mark this patch, as needs a new revision for now.

Cheers,

Eelco


> ---
>  debian/automake.mk | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/debian/automake.mk b/debian/automake.mk
> index 7b2afafae1a2..ebdcb022b3ef 100644
> --- a/debian/automake.mk
> +++ b/debian/automake.mk
> @@ -113,6 +113,11 @@ CLEANFILES += debian/control
>  debian: debian/copyright debian/control
>  .PHONY: debian
>
> +if DPDK_NETDEV
> +     DEB_BUILD_OPTIONS ?= "nocheck parallel=`nproc`"
> +else
> +     DEB_BUILD_OPTIONS ?= "nocheck parallel=`nproc` nodpdk"
> +endif
>
>  debian-deb: debian
>       @if test X"$(srcdir)" != X"$(top_builddir)"; then                       
> \
> @@ -123,10 +128,5 @@ debian-deb: debian
>       $(update_deb_copyright)
>       $(update_deb_control)
>       $(AM_V_GEN) fakeroot debian/rules clean
> -if DPDK_NETDEV
> -     $(AM_V_GEN) DEB_BUILD_OPTIONS="nocheck parallel=`nproc`" \
> +     $(AM_V_GEN) DEB_BUILD_OPTIONS="${DEB_BUILD_OPTIONS}" \
>               fakeroot debian/rules binary
> -else
> -     $(AM_V_GEN) DEB_BUILD_OPTIONS="nocheck parallel=`nproc` nodpdk" \
> -             fakeroot debian/rules binary
> -endif
> -- 
> 2.46.1
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to