hooks: Add function to check if a hook exists
authorAaron Schrab <aaron@schrab.com>
Sun, 13 Jan 2013 05:17:02 +0000 (00:17 -0500)
committerJunio C Hamano <gitster@pobox.com>
Mon, 14 Jan 2013 17:25:40 +0000 (09:25 -0800)
commit5a7da2dca166fab74f4514697e26dd80e79933f5
treed00b416154fd3790e7cf23592f6d8815dec8eb4b
parent94702dd1ac27618b60198f7c8bceafaaaf7743e2
hooks: Add function to check if a hook exists

Create find_hook() function to determine if a given hook exists and is
executable.  If it is, the path to the script will be returned,
otherwise NULL is returned.

This encapsulates the tests that are used to check for the existence of
a hook in one place, making it easier to modify those checks if that is
found to be necessary.  This also makes it simple for places that can
use a hook to check if a hook exists before doing, possibly lengthy,
setup work which would be pointless if no such hook is present.

The returned value is left as a static value from get_pathname() rather
than a duplicate because it is anticipated that the return value will
either be used as a boolean, immediately added to an argv_array list
which would result in it being duplicated at that point, or used to
actually run the command without much intervening work.  Callers which
need to hold onto the returned value for a longer time are expected to
duplicate the return value themselves.

Signed-off-by: Aaron Schrab <aaron@schrab.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/commit.c
builtin/receive-pack.c
run-command.c
run-command.h