diff options
| author | Stefan Beller <sbeller@google.com> | 2016-05-31 16:59:33 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-06-01 11:31:49 -0700 |
| commit | b0f4b4084626ab0f87611acd89030e8bdf0b850e (patch) | |
| tree | 7e4419d2415305c02bb85eb300389176000835ee /builtin | |
| parent | 60bd4b1c513bb652cdffad44382046ca872140eb (diff) | |
| download | git-b0f4b4084626ab0f87611acd89030e8bdf0b850e.tar.gz | |
submodule--helper: offer a consistent APIsb/submodule-helper-list-signal-unmatch-via-exit-status
In 48308681 (2016-02-29, git submodule update: have a dedicated helper
for cloning), the helper communicated errors back only via exit code,
and dance with printing '#unmatched' in case of error was left to
git-submodule.sh as it uses the output of the helper and pipes it into
shell commands. This change makes the helper consistent by never
printing '#unmatched' in the helper but always handling these piping
issues in the actual shell script.
Signed-off-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/submodule--helper.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c index 8da263f0b0..f0b2c4fe0b 100644 --- a/builtin/submodule--helper.c +++ b/builtin/submodule--helper.c @@ -287,10 +287,8 @@ static int module_list(int argc, const char **argv, const char *prefix) argc = parse_options(argc, argv, prefix, module_list_options, git_submodule_helper_usage, 0); - if (module_list_compute(argc, argv, prefix, &pathspec, &list) < 0) { - printf("#unmatched\n"); + if (module_list_compute(argc, argv, prefix, &pathspec, &list) < 0) return 1; - } for (i = 0; i < list.nr; i++) { const struct cache_entry *ce = list.entries[i]; |