All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: Anatol Pomozov <anatol.pomozov@gmail.com>, git@vger.kernel.org
Subject: Re: Incorrect git-blame result if I use full path to file
Date: Sun, 02 Dec 2007 18:40:36 -0800	[thread overview]
Message-ID: <7v4pf0sdp7.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <20071203022729.GD8322@coredump.intra.peff.net> (Jeff King's message of "Sun, 2 Dec 2007 21:27:29 -0500")

Jeff King <peff@peff.net> writes:

> IOW, it's not intended for users to use absolute paths in this way.
> However, the results for git-blame are obviously quite confusing. It
> might be worth fixing, but I suspect there are many more such traps
> waiting in other commands. I wonder if it would make sense to reject
> pathspecs starting with '/' entirely, which would at least give us a
> saner error message (and I can't think of a time when such a pathspec
> would be useful)?

All correct, except...

> Even more useful would be to convert
> /path/to/repo/file to 'file' internally.

... that might help "cut & paste from file manager" people, and I think
we had comment session for such a patch recently on the list.

Sorry, but I lost track of that the current status of that patch.  Did
it die?

  reply	other threads:[~2007-12-03  2:41 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-03  0:52 Incorrect git-blame result if I use full path to file Anatol Pomozov
2007-12-03  2:19 ` Junio C Hamano
2007-12-03  2:28   ` Jeff King
2007-12-03 17:26   ` Linus Torvalds
2007-12-03 18:09     ` Johannes Schindelin
2007-12-03 18:13       ` Linus Torvalds
2007-12-03 18:19         ` Linus Torvalds
2007-12-03  2:27 ` Jeff King
2007-12-03  2:40   ` Junio C Hamano [this message]
2007-12-03  2:49     ` Jeff King
2007-12-03  6:55       ` Robin Rosenberg
2007-12-03 20:53         ` [PATCH] Make Git accept absolute path names for files within the work tree Robin Rosenberg
2007-12-03 23:03           ` Junio C Hamano
2007-12-04  1:43           ` Jeff King
2007-12-04  2:17             ` Johannes Schindelin
2007-12-04  6:42               ` Robin Rosenberg
2007-12-04 11:50                 ` Johannes Schindelin
2007-12-04 15:59                   ` Linus Torvalds
2007-12-04 22:08                     ` Jeff King
2007-12-04 22:52                       ` Linus Torvalds
2007-12-06  6:12                         ` Jeff King

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7v4pf0sdp7.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=anatol.pomozov@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=peff@peff.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.