diff options
| author | Stefan Beller <sbeller@google.com> | 2017-03-25 00:36:08 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-03-27 16:48:19 (GMT) |
| commit | 5c896f7c3ec69f017d1e1be4164d558918f3ae4c (patch) | |
| tree | 3063c5e74a79494457e286c0022e7615c7a9e9be | |
| parent | fcecf0b968b5e262200426ccf1d0b82495c261fe (diff) | |
| download | git-5c896f7c3ec69f017d1e1be4164d558918f3ae4c.zip git-5c896f7c3ec69f017d1e1be4164d558918f3ae4c.tar.gz git-5c896f7c3ec69f017d1e1be4164d558918f3ae4c.tar.bz2 | |
submodule.c: stricter checking for submodules in is_submodule_modified
By having a stricter check in the superproject we catch errors earlier,
instead of spawning a child process to tell us.
Signed-off-by: Stefan Beller <sbeller@google.com>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | submodule.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/submodule.c b/submodule.c index 5865795..fa21c7b 100644 --- a/submodule.c +++ b/submodule.c @@ -1052,11 +1052,12 @@ unsigned is_submodule_modified(const char *path, int ignore_untracked) git_dir = read_gitfile(buf.buf); if (!git_dir) git_dir = buf.buf; - if (!is_directory(git_dir)) { + if (!is_git_directory(git_dir)) { + if (is_directory(git_dir)) + die(_("'%s' not recognized as a git repository"), git_dir); strbuf_release(&buf); /* The submodule is not checked out, so it is not modified */ return 0; - } strbuf_reset(&buf); |