From: Johannes Sixt <j6t@kdbg.org>
Date: Fri, 4 Feb 2011 08:41:58 +0000 (+0100)
Subject: start_command: flush buffers in the WIN32 code path as well
X-Git-Tag: v1.7.4.1~18
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=13af8cbd6af3f572a5c6c2fb7adc8fc7c408a3e2;p=git.git

start_command: flush buffers in the WIN32 code path as well

The POSIX code path did The Right Thing already, but we have to do the same
on Windows.

This bug caused failures in t5526-fetch-submodules, where the output of
'git fetch --recurse-submodules' was in the wrong order.

Debugged-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/run-command.c b/run-command.c
index 2a1041ef6..f91e446c8 100644
--- a/run-command.c
+++ b/run-command.c
@@ -194,6 +194,7 @@ fail_pipe:
 	}
 
 	trace_argv_printf(cmd->argv, "trace: run_command:");
+	fflush(NULL);
 
 #ifndef WIN32
 {
@@ -201,7 +202,6 @@ fail_pipe:
 	if (pipe(notify_pipe))
 		notify_pipe[0] = notify_pipe[1] = -1;
 
-	fflush(NULL);
 	cmd->pid = fork();
 	if (!cmd->pid) {
 		/*