run-command.c: introduce trace_run_command()

This is the same as the old code that uses trace_argv_printf() in
run-command.c. This function will be improved in later patches to
print more information from struct child_process.

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/run-command.c b/run-command.c
index 31fc5ea86eb6a3fe3650b8ac50f19f8621d90b9d..eadba8cb0d510aab2357b0998286a0c0aac4c2b0 100644 (file)
--- a/run-command.c
+++ b/run-command.c
@@ -6,6 +6,7 @@
 #include "thread-utils.h"
 #include "strbuf.h"
 #include "string-list.h"
 #include "thread-utils.h"
 #include "strbuf.h"
 #include "string-list.h"

+#include "quote.h"

 
 void child_process_init(struct child_process *child)
 {
 
 void child_process_init(struct child_process *child)
 {


@@ -556,6 +557,20 @@ static int wait_or_whine(pid_t pid, const char *argv0, int in_signal)
        return code;
 }
 
        return code;
 }
 

+static void trace_run_command(const struct child_process *cp)
+{
+       struct strbuf buf = STRBUF_INIT;
+
+       if (!trace_want(&trace_default_key))
+               return;
+
+       strbuf_addf(&buf, "trace: run_command:");
+       sq_quote_argv_pretty(&buf, cp->argv);
+
+       trace_printf("%s", buf.buf);
+       strbuf_release(&buf);
+}
+

 int start_command(struct child_process *cmd)
 {
        int need_in, need_out, need_err;
 int start_command(struct child_process *cmd)
 {
        int need_in, need_out, need_err;


@@ -624,7 +639,8 @@ int start_command(struct child_process *cmd)
                cmd->err = fderr[0];
        }
 
                cmd->err = fderr[0];
        }
 

-       trace_argv_printf(cmd->argv, "trace: run_command:");
+       trace_run_command(cmd);
+

        fflush(NULL);
 
 #ifndef GIT_WINDOWS_NATIVE
        fflush(NULL);
 
 #ifndef GIT_WINDOWS_NATIVE