[ovs-dev] [PATCH] appveyor: Build windows kernel datapath.

[Eitan Eliahu]

Guru, driver build is very short (less than a min), It uses precompiled headers. Let's have the debug build until we will have it as an option.

For user mode to take advantage on precompiled header, there is work to be dome in arranging the header files.

Thank you for taking the time!
Eitan  

-----Original Message-----
From: Gurucharan Shetty [mailto:shettyg at nicira.com] 
Sent: Friday, February 06, 2015 7:39 AM
To: Eitan Eliahu
Cc: Nithin Raju; dev at openvswitch.org; Gurucharan Shetty
Subject: Re: [ovs-dev] [PATCH] appveyor: Build windows kernel datapath.

> Would be nice to have an option for Release or debug build. User mode code too.
Right now, we are not deploying the created executables anywhere.
Appveyor does have that option. We can probably enhance it once we have a in-tree windows installer.

On the release/debug question, the only option is to do both - which will increase the build time. Increase of build time means that pre-checkin check through appveyor will be slow and people may not actually use it.

On the question of the pre-compiled headers, it is likely not going to help the userspace build because it is done one file at a time. I will nevertheless investigate.

>
> -----Original Message-----
> From: dev [mailto:dev-bounces at openvswitch.org] On Behalf Of Nithin 
> Raju
> Sent: Thursday, February 05, 2015 10:39 PM
> To: Gurucharan Shetty
> Cc: dev at openvswitch.org; Gurucharan Shetty
> Subject: Re: [ovs-dev] [PATCH] appveyor: Build windows kernel datapath.
>
>> +- C:\MinGW\msys\1.0\bin\bash -lc "cd /c/openvswitch && ./configure CC=build-aux/cccl LD=\"`which link`\" LIBS=-lws2_32 --with-pthread=C:/pthreads-win32/Pre-built.2 --with-openssl=C:/OpenSSL-Win32 --with-vstudioddk=\"Win8.1 Release\”"
>
>
>
> Thanks for the patch.
>
>
>
> Only comment I had was to was to prefer to use "Win8.1 Debug” rather than “Win8.1 Release”. It doesn’t make any practical difference though, but it might be best to do “Debug” builds rather than “Release” in a build-test setup.
>
>
>
> Acked-by: Nithin Raju <nithin at vmware.com> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailman_listinfo_dev&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=CWsgHUxi6ExLXY798tmo3LJ4e3geGYp56lkcH-5cLCY&m=MQliY0lervWD1h0IE-vbsFegcFKLa9zmypMkSUENsKU&s=xwhEJXSZhbRRLtHYTw6veobMVUDNCK01EUqRgvoeOlw&e=