Re: [PATCH] pull: mention "pull", not "fetch" in the error message.
- From
Michael J Gruber <git@drmicha.warpmail.net>
- Date
- Mar 26, 2010, 14:18 UTC
- Message-ID
- <4BACC241.4010608@drmicha.warpmail.net>
- In-Reply-To
- <1269610699-13176-1-git-send-email-Matthieu.Moy@imag.fr>
Matthieu Moy venit, vidit, dixit 26.03.2010 14:38:
> For newbies who've just been taught "git push", the error message
Even though newbies don't read git.git's commit messages, you may want to say "pull" here ;)
One way or the other, nothing beats the error messages in ident.c ;)
Show 42 quoted lines
> "Where do you want to fetch from today?" is indeed confusing. Change it
> to "Where do you want to pull from today?" in case fetch was called from
> pull.
>
> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
> ---
> builtin/fetch.c | 6 +++++-
> git-pull.sh | 3 ++-
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/builtin/fetch.c b/builtin/fetch.c
> index 957be9f..f3246f5 100644
> --- a/builtin/fetch.c
> +++ b/builtin/fetch.c
> @@ -842,8 +842,12 @@ static int fetch_one(struct remote *remote, int argc, const char **argv)
> int ref_nr = 0;
> int exit_code;
>
> + char *cmd = getenv("GIT_USER_COMMAND");
> + if (cmd == NULL || cmd[0] == '\0')
> + cmd = "fetch";
> +
> if (!remote)
> - die("Where do you want to fetch from today?");
> + die("Where do you want to %s from today?", cmd);
>
> transport = transport_get(remote, NULL);
> transport_set_verbosity(transport, verbosity, progress);
> diff --git a/git-pull.sh b/git-pull.sh
> index 1a4729f..abc233b 100755
> --- a/git-pull.sh
> +++ b/git-pull.sh
> @@ -216,7 +216,8 @@ test true = "$rebase" && {
> done
> }
> orig_head=$(git rev-parse -q --verify HEAD)
> -git fetch $verbosity $progress --update-head-ok "$@" || exit 1
> +GIT_USER_COMMAND=pull \
> + git fetch $verbosity $progress --update-head-ok "$@" || exit 1
>
> curr_head=$(git rev-parse -q --verify HEAD)
> if test -n "$orig_head" && test "$curr_head" != "$orig_head"