]> git.ipfire.org Git - thirdparty/git.git/commit - builtin/grep.c
builtin/grep.c: respect 'submodule.recurse' option
authorStefan Beller <sbeller@google.com>
Thu, 1 Jun 2017 00:30:48 +0000 (17:30 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 1 Jun 2017 01:36:36 +0000 (10:36 +0900)
commit9071c078af637573a3068c6920dd4ceff6a5a69f
tree71590a61a533ec630a6af98f411725d5cfdb1116
parent046b48239eca34425909330e59da57f5fd421bdc
builtin/grep.c: respect 'submodule.recurse' option

In builtin/grep.c we parse the config before evaluating the command line
options. This makes the task of teaching grep to respect the new config
option 'submodule.recurse' very easy by just parsing that option.

As an alternative I had implemented a similar structure to treat
submodules as the fetch/push command have, including
* aligning the meaning of the 'recurse_submodules' to possible submodule
  values RECURSE_SUBMODULES_* as defined in submodule.h.
* having a callback to parse the value and
* reacting to the RECURSE_SUBMODULES_DEFAULT state that was the initial
  state.

However all this is not needed for a true boolean value, so let's keep
it simple. However this adds another place where "submodule.recurse" is
parsed.

Signed-off-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/grep.c
t/t7814-grep-recurse-submodules.sh