On Fri, Dec 9, 2016 at 11:21 AM, Russell Bryant <[email protected]> wrote:
> > > On Thu, Dec 8, 2016 at 8:31 PM, Andy Zhou <[email protected]> wrote: > >> OVN is packaged with openvswitch-fedora.spec.in, but not with >> openvswitch.spec.in. Remove OVN files from openvswitch.spec.in >> builds to make rpm build happy. >> >> Signed-off-by: Andy Zhou <[email protected]> >> > > Acked-by: Russell Bryant <[email protected]> > Thanks, I will push it soon. > > but 1 minor suggestion ... > > >> --- >> rhel/openvswitch.spec.in | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in >> index 9f6549a..14f6e98 100644 >> --- a/rhel/openvswitch.spec.in >> +++ b/rhel/openvswitch.spec.in >> @@ -103,7 +103,8 @@ rm \ >> $RPM_BUILD_ROOT/usr/bin/ovn-* \ >> $RPM_BUILD_ROOT/usr/share/man/man?/ovn-* \ >> $RPM_BUILD_ROOT/usr/share/openvswitch/ovn-* \ >> - $RPM_BUILD_ROOT/usr/share/openvswitch/scripts/ovn-* >> + $RPM_BUILD_ROOT/usr/share/openvswitch/scripts/ovn-* \ >> + $RPM_BUILD_ROOT/usr/share/openvswitch/scripts/ovndb-* >> >> > or just "ovn*" instead of 2 lines ... > Sure, why not. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
