[PATCH] run_command(): respect GIT_TRACE

Johannes Schindelin· Jul 7, 2008, 13:41 UTC · lore

When GIT_TRACE is set, the user is most likely wanting to see an external command that is about to be executed.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 run-command.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Show changes to run-command.c +2 −0

diff --git a/run-command.c b/run-command.c
index 2ce8c2b..6e29fdf 100644
--- a/run-command.c
+++ b/run-command.c
@@ -65,6 +65,8 @@ int start_command(struct child_process *cmd)
 		cmd->err = fderr[0];
 	}
 
+	trace_argv_printf(cmd->argv, "trace: run_command:");
+
 #ifndef __MINGW32__
 	cmd->pid = fork();
 	if (!cmd->pid) {

-- 
1.5.6.2.402.g95b5ab.dirty

Junio C Hamano· Jul 7, 2008, 20:00 UTC · re: Johannes Schindelin · lore

Re: [PATCH] run_command(): respect GIT_TRACE

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> When GIT_TRACE is set, the user is most likely wanting to see an external
> command that is about to be executed.

I haven't checked to see if none of the callers of start_command() assumes the current behaviour and is printing this information itself instead, but other than that, your justification feels very sensible.

Thanks.

Show 20 quoted lines

> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
>  run-command.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/run-command.c b/run-command.c
> index 2ce8c2b..6e29fdf 100644
> --- a/run-command.c
> +++ b/run-command.c
> @@ -65,6 +65,8 @@ int start_command(struct child_process *cmd)
>  		cmd->err = fderr[0];
>  	}
>  
> +	trace_argv_printf(cmd->argv, "trace: run_command:");
> +
>  #ifndef __MINGW32__
>  	cmd->pid = fork();
>  	if (!cmd->pid) {
> -- 
> 1.5.6.2.402.g95b5ab.dirty

patchrun_command(): respect GIT_TRACE