]> git.ipfire.org Git - thirdparty/git.git/commit - contrib/buildsystems/Generators/Vcxproj.pm
contrib/buildsystems: fix Visual Studio Debug configuration
authorAlexandr Miloslavskiy <alexandr.miloslavskiy@syntevo.com>
Mon, 23 Sep 2019 08:28:35 +0000 (01:28 -0700)
committerJunio C Hamano <gitster@pobox.com>
Sat, 28 Sep 2019 04:43:36 +0000 (13:43 +0900)
commitfe0ed5d5e9d726d195758f7ae9b00e61399d8319
tree20db43c4cc7538ef8234506c0da5a6ff29433aed
parent5fa0f5238b0cd46cfe7f6fa76c3f526ea98148d9
contrib/buildsystems: fix Visual Studio Debug configuration

Even though Debug configuration builds, the resulting build is incorrect
in a subtle way: it mixes up Debug and Release binaries, which in turn
causes hard-to-predict bugs.

In my case, when git calls iconv library, iconv sets 'errno' and git
then tests it, but in Debug and Release CRT those 'errno' are different
memory locations.

This patch addresses 3 connected bugs:
1) Typo in '\(Configuration)'. As a result, Debug configuration
   condition is always false and Release path is taken instead.
2) Regexp that replaced 'zlib.lib' with 'zlibd.lib' was only affecting
   the first occurrence. However, some projects have it listed twice.
   Previously this bug was hidden, because Debug path was never taken.
   I decided that avoiding double -lz in makefile is fragile and I'd
   better replace all occurrences instead.
3) In Debug, 'libcurl-d.lib' should be used instead of 'libcurl.lib'.
   Previously this bug was hidden, because Debug path was never taken.

Signed-off-by: Alexandr Miloslavskiy <alexandr.miloslavskiy@syntevo.com>
Acked-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
contrib/buildsystems/Generators/Vcxproj.pm