On Wed, May 22, 2013 at 6:39 AM, Ramkumar Ramachandra
<artagnon@xxxxxxxxx> wrote:
> Currently, when we try to resolve @{<N>} or @{<date>} when the reflog
> doesn't go back far enough, we get errors like:
>
> # on branch master
> $ git show @{10000}
> fatal: Log for '' only has 7 entries.
>
> $ git show @{10000.days.ago}
> warning: Log for '' only goes back to Tue, 21 May 2013 14:14:45 +0530.
> ...
>
> # detached HEAD case
> $ git show @{10000}
> fatal: Log for '' only has 2005 entries.
>
> $ git show master@{10000}
> fatal: Log for 'master' only has 7 entries.
>
> The empty string '' is ugly, inconsistent, and failing to convey
s/failing/fails/
> information about whose logs we are inspecting. Change this so that we
> get:
>
> # on branch master
> $ git show @{10000}
> fatal: Log for 'master' only has 7 entries.
>
> $ git show @{10000.days.ago}
> warning: Log for 'master' only goes back to Tue, 21 May 2013 14:14:45 +0530.
> ...
>
> # detached HEAD case
> $ git show @{10000}
> fatal: Log for 'HEAD' only has 2005 entries.
>
> $ git show master@{10000}
> fatal: Log for 'master' only has 7 entries.
>
> Simple, consistent, and informative; suitable for output even from
> plumbing commands like rev-parse.
>
> Signed-off-by: Ramkumar Ramachandra <artagnon@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html