It depends.

When compiling with VS2013 it is default. I compiled with VS2015 and for some 
reason it got compiled with MD because I looked with dupmbin/dependency walker 
and msvcr* was needed by the binary itself.

The idea would be it is ok to have the flag so we know exactly how the binary 
will end up. I don't know if forcing this flag is a good idea overall because 
we should probably have switch to specify it, however for now I think it would 
suffice and we can add a configure argument later on for it.

Alin.

> -----Mesaj original-----
> De la: Nithin Raju [mailto:nit...@vmware.com]
> Trimis: Tuesday, March 1, 2016 1:17 AM
> Către: Alin Serdean <aserd...@cloudbasesolutions.com>;
> dev@openvswitch.org
> Subiect: Re: [ovs-dev] [PATCH] Build Windows include runtime
> 
> Alin,
> Did you notice any change in the size of the binary? Eg. Ovs-vswitchd.exe?
> 
> With or without your change, the binary was the same size. I am wondering if
> the default is /MT?
> 
> Thanks,
> -- Nithin
> 
> -----Original Message-----
> From: dev <dev-boun...@openvswitch.org> on behalf of Alin Serdean
> <aserd...@cloudbasesolutions.com>
> Date: Friday, February 26, 2016 at 5:59 AM
> To: "dev@openvswitch.org" <dev@openvswitch.org>
> Subject: [ovs-dev] [PATCH] Build Windows include runtime
> 
> >This patch adds the Visual Studio runtime to the binaries by adding the
> >code generation compile flags: MT (release version) and MTd (debug
> >version).
> >
> >More on the subject can be found under:
> >https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__msdn.microsoft.com
> >_en
> >-2Dus_library_2kzt1wy3-2528v-3Dvs.120-
> 2529.aspx&d=BQIGaQ&c=Sqcl0Ez6M0X8
> >aeM
> >67LKIiDJAXVeAw-YihVMNtXt-
> uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3Jq
> >V80
> >&m=AzYieTQjHhOA7Cdg1MieX5MNzka2gaKrSjqul1VGOjk&s=zAtoC2irAQ3T
> sVgo02i5By
> >9y7
> >1HCwJb44BNgPubHnLA&e=
> >
> >This helps shipping binaries and guaranteeing the runtime uses the
> >specific version
> >
> >Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com>
> >---
> > build-aux/cccl | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> >diff --git a/build-aux/cccl b/build-aux/cccl index afa0a6b..93f9c50
> >100644
> >--- a/build-aux/cccl
> >+++ b/build-aux/cccl
> >@@ -90,11 +90,11 @@ EOF
> >         ;;
> >
> >     -O0)
> >-        clopt="$clopt ${slash}Od ${slash}D_DEBUG"
> >+        clopt="$clopt ${slash}Od ${slash}D_DEBUG ${slash}MTd"
> >         ;;
> >
> >     -O2)
> >-        clopt="$clopt ${slash}O2"
> >+        clopt="$clopt ${slash}O2 ${slash}MT"
> >         ;;
> >
> >     -L*)
> >--
> >1.9.5.msysgit.0
> >_______________________________________________
> >dev mailing list
> >dev@openvswitch.org
> >https://urldefense.proofpoint.com/v2/url?u=http-
> 3A__openvswitch.org_mai
> >lma
> >n_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-
> YihVMNtXt-uEs&r
> >=pN
> >HQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=AzYieTQjHhOA7Cdg1
> MieX5MNzka
> >2ga
> KrSjqul1VGOjk&s=Ium2WLera4JMpqWARWdfv_gFHhMQyMuWdmM7NR5J
> QVU&e=

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to