Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

Bleep bloop. Greetings Numan Siddique, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 89 characters long (recommended limit is 79) #72 FILE: .travis/linux-build.s

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

On Thu, Aug 1, 2019 at 7:49 PM Ilya Maximets wrote: > On 01.08.2019 16:21, Numan Siddique wrote: > > > > > > On Thu, Aug 1, 2019 at 6:33 PM Ilya Maximets > wrote: > > > > > On Thu, Aug 1, 2019 at 5:57 PM Lucas Alvares Gomes gmail.com > > >

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

On 01.08.2019 16:21, Numan Siddique wrote: > > > On Thu, Aug 1, 2019 at 6:33 PM Ilya Maximets > wrote: > > > On Thu, Aug 1, 2019 at 5:57 PM Lucas Alvares Gomes gmail.com > > > wrote: > > > >> Thanks Numan, > >> > >> I've

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

On Thu, Aug 1, 2019 at 6:33 PM Ilya Maximets wrote: > > On Thu, Aug 1, 2019 at 5:57 PM Lucas Alvares Gomes gmail.com> > > wrote: > > > >> Thanks Numan, > >> > >> I've manually tested it (using the snippet example in the commit > >> message) locally in an env of mine and it works. > >> > > > > Th

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

> On Thu, Aug 1, 2019 at 5:57 PM Lucas Alvares Gomes > wrote: > >> Thanks Numan, >> >> I've manually tested it (using the snippet example in the commit >> message) locally in an env of mine and it works. >> > > Thanks for the review. Right now this patch is failing in travis CI. It > runs "make

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

On Thu, Aug 1, 2019 at 5:57 PM Lucas Alvares Gomes wrote: > Thanks Numan, > > I've manually tested it (using the snippet example in the commit > message) locally in an env of mine and it works. > Thanks for the review. Right now this patch is failing in travis CI. It runs "make distcheck" and it

Re: [ovs-dev] [PATCH ovn] Build OVN using external OVS directory

Thanks Numan, I've manually tested it (using the snippet example in the commit message) locally in an env of mine and it works. On Wed, Jul 31, 2019 at 7:49 PM wrote: > > From: Numan Siddique > > With this patch we have to configure OVN to refer to external OVS source/build > directory instead

[ovs-dev] [PATCH ovn] Build OVN using external OVS directory

From: Numan Siddique With this patch we have to configure OVN to refer to external OVS source/build directory instead of the ovs subtree. The new configuration options added are: * --with-ovs-source=/path/to/ovs/source/dir * --with-ovs-build=/path/to/ovs/build/dir Before configuring OVN, user