[ovs-dev] [PATCH ovn v10.1 0/6] Add DDlog implementation of ovn-northd

Ben Pfaff blp at ovn.org
Tue Feb 23 17:44:50 UTC 2021


On Tue, Feb 23, 2021 at 07:38:12PM +0530, Numan Siddique wrote:
> On Tue, Feb 23, 2021 at 4:10 PM Numan Siddique <numans at ovn.org> wrote:
> >
> > On Fri, Feb 19, 2021 at 4:10 AM Ben Pfaff <blp at ovn.org> wrote:
> > >
> > > This passed in the ovsrobot CI:
> > > https://github.com/ovsrobot/ovn/actions/runs/579307688
> > >
> > > I think that we have arrived at consensus to push this to ovn master
> > > after branching happens tomorrow.  Please, let me know if I
> > > misunderstood!
> >
> > I think understanding is correct.  I think Mark needs to create the branch.
> 
> Hi Ben,
> 
> I am trying to compile with ddlog configured, but the compilation is
> failing for me
> for some reason. Can you please help me with this ?

Oops.  I made a dumb typo.  It didn't show up in my testing because I
guess a -j12 build typically built OVN_Northbound.dl and
OVN_Southbound.dl early enough.  Here's the fix:

diff --git a/northd/automake.mk b/northd/automake.mk
index 17c394c82082..d64d1d197c77 100644
--- a/northd/automake.mk
+++ b/northd/automake.mk
@@ -29,7 +29,7 @@ ddlog_sources = \
 	northd/ovn.dl \
 	northd/ovn.rs \
 	northd/helpers.dl
-ddlog_nodist_source = \
+ddlog_nodist_sources = \
 	northd/OVN_Northbound.dl \
 	northd/OVN_Southbound.dl
 


More information about the dev mailing list