Patrick Steinhardt <p...@pks.im> writes:
> Due to various components of the URI being stripped off it may
> happen that we fail to guess a directory name. We currently error
> out with a message that it is impossible to create the working
> tree '' in such cases. Instead, error out early with a sensible
> error message hinting that a directory name should be specified
> manually on the command line.
Sounds like a sensible thing to do.
>
> Signed-off-by: Patrick Steinhardt <p...@pks.im>
> ---
> builtin/clone.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/builtin/clone.c b/builtin/clone.c
> index 7d93e13..5834978 100644
> --- a/builtin/clone.c
> +++ b/builtin/clone.c
> @@ -214,6 +214,10 @@ static char *guess_dir_name(const char *repo, int
> is_bundle, int is_bare)
> len = end - start;
> strip_suffix_mem(start, &len, is_bundle ? ".bundle" : ".git");
>
> + if (!len || (len == 1 && *start == '/'))
> + die("No directory name could be guessed.\n"
> + "Please specify a directory on the command line");
> +
> if (is_bare)
> dir = xstrfmt("%.*s.git", (int)len, start);
> else
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
