<div dir="auto"><div><br><div class="gmail_extra"><div class="gmail_quote">On 28 Sep 2017 00:10, "Jeff McKenna" <<a href="mailto:jmckenna@gatewaygeomatics.com">jmckenna@gatewaygeomatics.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">On 2017-09-27 4:52 PM, Mateusz Loskot wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 27 September 2017 at 21:38, Even Rouault <<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Once again someone raised an issue about the MSVC runtime we use for DEBUG=1<br>
builds: <a href="https://trac.osgeo.org/gdal/ticket/7059" rel="noreferrer" target="_blank">https://trac.osgeo.org/gdal/ti<wbr>cket/7059</a><br>
<br>
[...]<br>
Personnaly I believe I've never used a Windows build, so I'm mostly<br>
indifferent<br>
<br>
Any opinion about using the debug MSVC runtime (/MDd) for DEBUG=1 builds<br>
instead of the current release runtime (/MD) ?<br>
</blockquote>
<br>
Even,<br>
<br>
Yesterday, I started thinking about adding an extra build configuration to NMAKE<br>
makefile, controlled via command line defines, to enable such proper/full debug<br>
build for Windows.<br>
<br>
IMO, debug builds w/o /MDd are somewhat unexpected or exotic.<br>
As one of Windows users and developers, I'd vote for the default<br>
MSVC configuration for debug builds of GDAL.<br>
<br>
This would also help a bit with binary packages for Visual C++,<br>
currently, it requires nmake.opt patching.<br>
<a href="https://github.com/Microsoft/vcpkg/pull/1879" rel="noreferrer" target="_blank">https://github.com/Microsoft/v<wbr>cpkg/pull/1879</a><br>
<br>
Best regards,<br>
<br>
</blockquote>
<br></div>
To clarify: Mateusz you mean that debug builds should be changed to /MDd, correct?  I somehow read your response as you also want to change the default build in nmake to be the debug build.  Or, maybe my brain is tired ha :)  -jeff</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Jeff, yes that is what I was trying to say,  to change the debug defaults only. </div><div dir="auto"><br></div><div dir="auto">Perhaps I shouldn't paste the link to vcpkg as the flags patched there are different, not relevant here. </div><div dir="auto"><br></div><div dir="auto">However, one may want to also package and offer debug binaries for MSVC like Python 3.5 or 3.6+, and /MDd would definitely be a must as that is the MSVC way. </div><div dir="auto"><br></div><div dir="auto">Mat</div><div dir="auto"></div></div>