Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-11-24 Thread Nithin Raju
On Nov 21, 2014, at 2:43 PM, Gurucharan Shetty wrote: > On Thu, Aug 28, 2014 at 6:49 AM, Alin Serdean > wrote: >> This commit adds to the automake build system the full build required >> by the forwarding extension solution. >> >> It will help a lot in the future CI to check the full build of

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-11-21 Thread Gurucharan Shetty
On Thu, Aug 28, 2014 at 6:49 AM, Alin Serdean wrote: > This commit adds to the automake build system the full build required > by the forwarding extension solution. > > It will help a lot in the future CI to check the full build of the project. > > To configure the forwarding extension to be built

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Ben Pfaff
king > directory does not contain a project or solution file. > > > > -Original Message- > From: Ben Pfaff [mailto:b...@nicira.com] > Sent: Thursday, August 28, 2014 8:50 AM > To: Alin Serdean > Cc: dev@openvswitch.org; Eitan Eliahu; Saurabh Shah; Nithin Raju >

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Eitan Eliahu
To: Alin Serdean Cc: dev@openvswitch.org; Eitan Eliahu; Saurabh Shah; Nithin Raju Subject: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild On Thu, Aug 28, 2014 at 01:49:24PM +, Alin Serdean wrote: > +if VSTUDIO_DDK > +ALL_LOCAL += ovsext_make > +ovsext_make: datapat

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Ben Pfaff
On Thu, Aug 28, 2014 at 01:49:24PM +, Alin Serdean wrote: > +if VSTUDIO_DDK > +ALL_LOCAL += ovsext_make > +ovsext_make: datapath-windows/ovsext.sln > + MSBuild.exe datapath-windows/ovsext.sln /target:Build > /property:Configuration="$(VSTUDIO_CONFIG)" Isn't it unusual to include the .exe

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Ben Pfaff
On Thu, Aug 28, 2014 at 03:37:23PM +, Saurabh Shah wrote: > > >This commit adds to the automake build system the full build required > >by the forwarding extension solution. > > > >It will help a lot in the future CI to check the full build of the > >project. > > > >To configure the forwarding

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Saurabh Shah
>This commit adds to the automake build system the full build required >by the forwarding extension solution. > >It will help a lot in the future CI to check the full build of the >project. > >To configure the forwarding extension to be built one could use the >following: >./configure CC=./build-a

[ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-28 Thread Alin Serdean
This commit adds to the automake build system the full build required by the forwarding extension solution. It will help a lot in the future CI to check the full build of the project. To configure the forwarding extension to be built one could use the following: ./configure CC=./build-aux/cccl L

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-27 Thread Alin Serdean
Hi Eithan, Will add them anyway for the moment. Alin. -Mesaj original- De la: Eitan Eliahu [mailto:elia...@vmware.com] Trimis: Wednesday, August 27, 2014 6:08 PM Către: Alin Serdean; Saurabh Shah; dev@openvswitch.org Subiect: RE: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-27 Thread Eitan Eliahu
rdean; Saurabh Shah; dev@openvswitch.org Subiect: RE: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild Hi Alin, Should we able to specify Debug or Release build when building with MSBuild.exe? Thanks, Eitan -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-27 Thread Alin Serdean
To: Saurabh Shah; dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild Hi Saurabh, At the end of the day today. Was on short PTO last week. Alin. -Mesaj original- De la: Saurabh Shah [mailto:ssaur...@vmware.com] Trimis: Wednesday, August 27, 2014 2:46

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-27 Thread Eitan Eliahu
: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild Hi Saurabh, At the end of the day today. Was on short PTO last week. Alin. -Mesaj original- De la: Saurabh Shah [mailto:ssaur...@vmware.com] Trimis: Wednesday, August 27, 2014 2:46 AM Către: Alin Serdean; dev@openvswitch.org Subiect

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-27 Thread Alin Serdean
Hi Saurabh, At the end of the day today. Was on short PTO last week. Alin. -Mesaj original- De la: Saurabh Shah [mailto:ssaur...@vmware.com] Trimis: Wednesday, August 27, 2014 2:46 AM Către: Alin Serdean; dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-26 Thread Saurabh Shah
Hey Alin, Are you planning on re-spinning this patch? Saurabh >This commit adds to the automake build system the full build required by >the >forwarding extension solution. > >It will help a lot in the future CI to check the full build of the >project. > >Also the documentation was updated. > >Si

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-20 Thread Alin Serdean
.0.18(0.48/3/2) > > Thanks, > Alin. > > -Mesaj original- > De la: Gurucharan Shetty [mailto:shet...@nicira.com] > Trimis: Tuesday, August 19, 2014 12:12 AM > Către: Alin Serdean > Cc: dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-19 Thread Gurucharan Shetty
gt; Alin. > > -Mesaj original- > De la: Gurucharan Shetty [mailto:shet...@nicira.com] > Trimis: Tuesday, August 19, 2014 12:12 AM > Către: Alin Serdean > Cc: dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild > > On Fri, Aug

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-18 Thread Alin Serdean
...@nicira.com] Trimis: Tuesday, August 19, 2014 12:12 AM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild On Fri, Aug 15, 2014 at 9:58 AM, Alin Serdean wrote: > Hey Guru, > > I can add a follow up on the after the version 2 t

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-18 Thread Gurucharan Shetty
our thoughts as well, > Alin. > > -Mesaj original- > De la: Gurucharan Shetty [mailto:shet...@nicira.com] > Trimis: Friday, August 15, 2014 7:52 PM > Către: Alin Serdean > Cc: dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-15 Thread Alin Serdean
...@nicira.com] Trimis: Friday, August 15, 2014 7:52 PM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild On Wed, Aug 13, 2014 at 8:13 AM, Alin Serdean wrote: > This commit adds to the automake build system the full build requi

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-15 Thread Gurucharan Shetty
On Wed, Aug 13, 2014 at 8:13 AM, Alin Serdean wrote: > This commit adds to the automake build system the full build required by the > forwarding extension solution. > > It will help a lot in the future CI to check the full build of the project. > > Also the documentation was updated. > > Signed-of

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-15 Thread Alin Serdean
Nithin, Thanks for the review I will send out a V2 as soon as possible. Thanks, Alin. -Mesaj original- De la: Nithin Raju [mailto:nit...@vmware.com] Trimis: Friday, August 15, 2014 3:52 AM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH] Add build of

Re: [ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-14 Thread Nithin Raju
Alin, Thanks for making this change. It is great that we can build via command line now. Some comments I had are: 1. Somewhere at the top we should mention that the 'forwarding extension' is a 'kernel driver'. For someone now familiar with Windows, it might make it more obvious as to which is t

[ovs-dev] [PATCH] Add build of ovsext.sln using MSBuild

2014-08-13 Thread Alin Serdean
This commit adds to the automake build system the full build required by the forwarding extension solution. It will help a lot in the future CI to check the full build of the project. Also the documentation was updated. Signed-off-by: Alin Gabriel Serdean --- INSTALL.Windows | 34 +