<div dir="auto"><div><br><div class="gmail_extra"><div class="gmail_quote">On 28 Sep 2017 00:08, "Carl Godkin" <<a href="mailto:cgodkin@gmail.com">cgodkin@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div class="quoted-text">On Wed, Sep 27, 2017 at 12:38 PM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>
<div style="font-family:monospace;font-size:9pt;font-weight:400;font-style:normal">
<p style="margin:0px;text-indent:0px">Hi,</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">For those who don't understand the title, you may skip this email ;-)</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">Once again someone raised an issue about the MSVC runtime we use for DEBUG=1 builds:</p>
<p style="margin:0px;text-indent:0px"><a href="https://trac.osgeo.org/gdal/ticket/7059" target="_blank">https://trac.osgeo.org/gdal/ti<wbr>cket/7059</a></p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">and in <a href="https://trac.osgeo.org/gdal/ticket/6384" target="_blank">https://trac.osgeo.org/gdal/ti<wbr>cket/6384</a> about the same issue, there are 4 similar tickets raised.</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">So it seems our default is a recurring problem.</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">Personnaly I believe I've never used a Windows build, so I'm mostly indifferent</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">Any opinion about using the debug MSVC runtime (/MDd) for DEBUG=1 builds instead of the current release runtime (/MD) ?</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px"><br></p></div></blockquote><div><br></div></div><div>Hi Even,</div><div><br></div><div>Since you ask, I can tell you that changing "/MD" to "/MDd" is one of just two remaining edits that I have to make to nmake.opt before I build a new version of GDAL.  </div></div></div></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Yup, me too and I think most GDAL as library users on Windows do it as well. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div></div></div></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>(By the way, the other item in nmake.opt that I can't override with an <span style="white-space:pre-wrap">EXT_NMAKE_OPT file is that we don't want to build with</span><br></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div>ODBC_SUPPORTED = 1 <br></div><div> </div><div>If that were changed to use the !IFNDEF ODBC_SUPPORTED then I wouldn't have to edit nmake.opt at all which would be very handy.)</div><div></div></div></div></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Then, why not submit a patch or pull request. I think it's good idea. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra">Mateusz </div></div></div>