From: Santi Béjar <santi@agolina.net>
Date: Wed, 29 Sep 2010 01:49:59 GMT
Subject: Re: [PATCH] checkout: add a space between the commit and "..."
Message-ID: <AANLkTimLZwG5dOy2+rP0eKTp0DmRnaTYf7h3OBsL5fhM@mail.gmail.com>
In-Reply-To: <20100928222332.GA28859@soprano.nvidia.com>

On Wed, Sep 29, 2010 at 12:23 AM, Aaron Plattner <aplattner@nvidia.com> wrote:
> Switching to a detached head prints something like
>
>  HEAD is now at 9d14017... dir.c: squelch false uninitialized memory warning

You should also change the message when going from one detached HEAD to another:

$ git co HEAD^
Previous HEAD position was 7bba3dd... Merge branch 'master' into next
HEAD is now at df51bf2... Merge branch 'dm/mergetool-vimdiff' into next

>
> These dots get selected when you double-click on the abbreviated
> commit hash, which makes it annoying to copy and paste.
>
> Add a space between the abbreviated commit and the dots, to improve
> copy and pasteability.

I would change it to mimic the output of "git commit", something like:

$ git co HEAD^
[HEAD was at 7bba3dd] Merge branch 'master' into next
[HEAD is now at df51bf2] Merge branch 'dm/mergetool-vimdiff' into next

HTH,
Santi