On Sat, Nov 03, 2007 at 10:21:53PM -0700, Junio C Hamano wrote:
> Mike Hommey <mh@xxxxxxxxxxxx> writes:
>
> > While 'git blame rev -- path' works, 'git blame rev path' didn't.
> >
> > Signed-off-by: Mike Hommey <mh@xxxxxxxxxxxx>
> > ---
> > builtin-blame.c | 4 ----
> > 1 files changed, 0 insertions(+), 4 deletions(-)
> >
> > diff --git a/builtin-blame.c b/builtin-blame.c
> > index aedc294..141287e 100644
> > --- a/builtin-blame.c
> > +++ b/builtin-blame.c
> > @@ -2294,10 +2294,6 @@ int cmd_blame(int argc, const char **argv, const char *prefix)
> > }
> > else if (i != argc - 1)
> > usage(blame_usage); /* garbage at end */
> > -
> > - if (!has_path_in_work_tree(path))
> > - die("cannot stat path %s: %s",
> > - path, strerror(errno));
> > }
> > }
> >
>
> Sorry but a NAK; at least limit the removal of the test only to a bare
> repository case.
I fail to see why this test is necessary here. The lack of file in the
working tree is caught in fake_working_tree_commit() anyways.
Mike
-
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
