Merge ~fnordahl/ubuntu/+source/openvswitch:extra-configure-opts into ~ubuntu-server-dev/ubuntu/+source/openvswitch:master

Proposed by Frode Nordahl
Status: Needs review
Proposed branch: ~fnordahl/ubuntu/+source/openvswitch:extra-configure-opts
Merge into: ~ubuntu-server-dev/ubuntu/+source/openvswitch:master
Diff against target: 24 lines (+7/-2)
1 file modified
debian/rules (+7/-2)
Reviewer Review Type Date Requested Status
James Page Pending
Review via email: mp+425699@code.launchpad.net

Description of the change

In preparing for proposing our packaging sources to the upstream Open vSwitch repository we pick this relevant change made to the upstream packaging sources.

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Hi Frode, welcome back

LGTM

If you do not mind (to reduce churn later on when merging with Debian) I'm picking this into the to-be-debians-ovs-based-on-ubuntu branches right away.

Unmerged commits

9da33bd... by Frode Nordahl

d/rules: Allow use of EXTRA_CONFIGURE_OPTS

Adapted from upstream commit [0].

0: https://github.com/openvswitch/ovs/commit/925a09477

Co-authored-by: Shahar Klein <email address hidden>
Signed-off-by: Frode Nordahl <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/rules b/debian/rules
2index 8000448..c9f4348 100755
3--- a/debian/rules
4+++ b/debian/rules
5@@ -25,14 +25,19 @@ override_dh_auto_configure:
6 cd _debian && ( \
7 test -e Makefile || \
8 ../configure --prefix=/usr --localstatedir=/var --enable-ssl \
9- --sysconfdir=/etc $(DATAPATH_CONFIGURE_OPTS))
10+ --sysconfdir=/etc \
11+ $(DATAPATH_CONFIGURE_OPTS) \
12+ $(EXTRA_CONFIGURE_OPTS) \
13+ )
14 ifneq (,$(filter i386 amd64 ppc64el arm64, $(DEB_HOST_ARCH)))
15 test -d _dpdk || mkdir _dpdk
16 cd _dpdk && ( \
17 test -e Makefile || \
18 ../configure --prefix=/usr --localstatedir=/var --enable-ssl \
19 --with-dpdk=shared --sysconfdir=/etc \
20- $(DATAPATH_CONFIGURE_OPTS))
21+ $(DATAPATH_CONFIGURE_OPTS) \
22+ $(EXTRA_CONFIGURE_OPTS) \
23+ )
24 endif
25
26

Subscribers

People subscribed via source and target branches