Skip to content

Commit

Permalink
clone: Fix error message for reference repository
Browse files Browse the repository at this point in the history
Do not report that an argument to clone's --reference option is not a
local directory.  Nothing checks for the existence or type of the path
as supplied by the user; checks are only done for particular contents of
the supposed directory, so we have no way to know the status of the
supplied path.  Telling the user that a directory doesn't exist when
that isn't actually known may lead him or her on the wrong path to
finding the problem.

Instead just state that the entered path is not a local repository which
is really all that is known about it.  It could be more helpful to state
the actual paths which were checked, but I believe that giving a good
description of that would be too verbose for a simple error message and
would be too dependent on implementation details.

Signed-off-by: Aaron Schrab <aaron@schrab.com>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Aaron Schrab authored and Junio C Hamano committed Apr 9, 2013
1 parent 52a3e01 commit 0658569
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion builtin/clone.c
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ static int add_one_reference(struct string_list_item *item, void *cb_data)
free(ref_git);
ref_git = ref_git_git;
} else if (!is_directory(mkpath("%s/objects", ref_git)))
die(_("reference repository '%s' is not a local directory."),
die(_("reference repository '%s' is not a local repository."),
item->string);

strbuf_addf(&alternate, "%s/objects", ref_git);
Expand Down

0 comments on commit 0658569

Please sign in to comment.