# run_hook: why special check for uncaught signal?

2 messages from 2009-06-30 to 2009-07-01. Participants: Johannes Sixt, Miklos Vajna.
Thread: https://gitlist.dev/t/19985

## Johannes Sixt, 2009-06-30 20:31

Subject: run_hook: why special check for uncaught signal?
Message-ID: <200906302231.45157.j6t@kdbg.org>
URL: https://gitlist.dev/e/200906302231.45157.j6t%40kdbg.org

```
Function run_hook (in run-command.c) has this check:

	ret = finish_command(&hook);
	if (ret == -ERR_RUN_COMMAND_WAITPID_SIGNAL)
		warning("%s exited due to uncaught signal", argv[0]);
	return ret;

The lines were moved to this location by ae98a008 (Move run_hook() from 
builtin-commit.c into run-command.c (libgit), 2009-01-16). Before this 
commit, the check existed in builtin-gc.c. It was introduced by bde30540 
(git-gc --auto: add pre-auto-gc hook, 2008-04-09).

Why is this check necessary?

I'm asking because I'm working on a modification that would remove 
ERR_RUN_COMMAND_WAITPID_SIGNAL.

-- Hannes

```

## Miklos Vajna, 2009-07-01 09:57

Subject: Re: run_hook: why special check for uncaught signal?
Message-ID: <20090701095751.GE10895@genesis.frugalware.org>
URL: https://gitlist.dev/e/20090701095751.GE10895%40genesis.frugalware.org
In-Reply-To: <200906302231.45157.j6t@kdbg.org>

```
On Tue, Jun 30, 2009 at 10:31:45PM +0200, Johannes Sixt <j6t@kdbg.org> wrote:
> Function run_hook (in run-command.c) has this check:
> 
> 	ret = finish_command(&hook);
> 	if (ret == -ERR_RUN_COMMAND_WAITPID_SIGNAL)
> 		warning("%s exited due to uncaught signal", argv[0]);
> 	return ret;
> 
> The lines were moved to this location by ae98a008 (Move run_hook() from 
> builtin-commit.c into run-command.c (libgit), 2009-01-16). Before this 
> commit, the check existed in builtin-gc.c. It was introduced by bde30540 
> (git-gc --auto: add pre-auto-gc hook, 2008-04-09).
> 
> Why is this check necessary?

I do not really remember, but I think I wanted to give some error
messages for each return code, and somehow left out
ERR_RUN_COMMAND_WAITPID, ERR_RUN_COMMAND_WAITPID_WRONG_PID and
ERR_RUN_COMMAND_WAITPID_NOEXIT.

> I'm asking because I'm working on a modification that would remove 
> ERR_RUN_COMMAND_WAITPID_SIGNAL.

That's fine, I think ideally run_hook() should call warning() for each
case which may be returned. Such functionality is already available in
builtin-receive-pack.c::hook_status() and unpack(), probably it would
make sense to introduce some run_command_strerror() to avoid code
duplication.

Sadly I don't have time to implement this in the near future.

```