Re: [PATCH] Change 'Deltifying objects' to 'Delta compressing objects'

From: Nicolas Pitre <nico@cam.org>
Date: Oct 19, 2007, 03:50 UTC
Message-ID: <alpine.LFD.0.9999.0710182340550.19446@xanadu.home>
In-Reply-To: <20071019033228.GA10697@coredump.intra.peff.net>

On Thu, 18 Oct 2007, Jeff King wrote:

Show 10 quoted lines

> On Thu, Oct 18, 2007 at 11:24:41PM -0400, Nicolas Pitre wrote:
> 
> > You usually get long lines that gets wrapped, so that means 3 lines of 
> > screen space for one updated branches.  Is the "66ffb04..4fa4d23" 
> > information really useful?  Might someone ever care?
> 
> I have used it occasionally when tracking repos to see what new commits
> have happened. Usually I use a separate branch to mark "what I've seen"
> (i.e., fetch, gitk origin..master, pull), but if it's a branch that I'm
> not actively tracking, the display is useful.

Maybe we should have a shortcut notation for <ref>@{1}..<ref> instead? I end up using that all the time since the fetch result has long scrolled off the screen when I want to look at what was fetched.

Usully this llooks like:

	git pull
	git log @{1}..

But using origin/master@{1}..origin/master is a bit cumbersome.

Show 8 quoted lines

> What is really useless in that line is the fact that _every_ ref is
> going to have the name of the remote, even though we only support
> fetching from one remote at a time. Perhaps something like:
> 
> Fetching from git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6
>  * refs/heads/origin: fast forward to branch 'master'
> 
> although that URL is almost a line by itself. :)

It is, therefore I'd skip "Fetching from " entirely.

Show 8 quoted lines

> Also, why do we abbreviate "refs/heads/master" from the remote, but we
> don't abbreviate refs/heads/origin for the local? Maybe something like:
> 
>   * local heads/origin -> remote heads/master (fast forward)
> 
> or for separate remote
> 
>   * local remotes/origin/master -> remote heads/master (fast forward)

That looks fine for a push. I'd say "remote foo -> local bar" for a fetch.

Nicolas