# [PATCH 0/3] show alias definitions in help

3 messages from 2008-02-24 to 2008-02-25. Participants: Jeff King, Daniel Barkalow.
Thread: https://gitlist.dev/t/12303

## Jeff King, 2008-02-24 22:16

Subject: [PATCH 0/3] show alias definitions in help
Message-ID: <20080224221559.GA31309@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20080224221559.GA31309%40coredump.intra.peff.net

```
This was requested during the freeze period, and I posted a "like this?"
patch with a promise to follow-up post-1.5.4. So here is the follow-up.

The objections to my quick patch were:

  - (Dscho) the parsing code was ugly and needed cleanup; the first
    patch refactors cmd_help to use parseopt
  - (Duy) help lookup should respect the usual "commands before alias"
    rule that we use for execution; the third patch takes this into
    account

      1/3: help: use parseopt
           This is parseopt plus general cleanup.

      2/3: make alias lookup a public, procedural function
           Another cleanup to factor out alias lookup from git.c (which
           we will also need in help now).

      help: respect aliases
           The actual patch.

 Makefile |    3 +-
 alias.c  |   22 +++++++++
 cache.h  |    2 +
 git.c    |   17 +------
 help.c   |  152 ++++++++++++++++++++++++++++++++++++--------------------------
 5 files changed, 118 insertions(+), 78 deletions(-)
 create mode 100644 alias.c

```

## Jeff King, 2008-02-24 22:23

Subject: Re: [PATCH 0/3] show alias definitions in help
Message-ID: <20080224222331.GA6712@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20080224222331.GA6712%40coredump.intra.peff.net
In-Reply-To: <20080224221559.GA31309@coredump.intra.peff.net>

```
On Sun, Feb 24, 2008 at 05:16:00PM -0500, Jeff King wrote:

>  Makefile |    3 +-
>  alias.c  |   22 +++++++++
>  cache.h  |    2 +
>  git.c    |   17 +------
>  help.c   |  152 ++++++++++++++++++++++++++++++++++++--------------------
>  5 files changed, 118 insertions(+), 78 deletions(-)
>  create mode 100644 alias.c

BTW, Daniel, I have used your --cover-letter work several times today,
and it is terrific. It has saved me a lot of error-prone cutting and
pasting of message-ids and commit titles. Thanks.

-Peff

```

## Daniel Barkalow, 2008-02-25 16:47

Subject: Re: [PATCH 0/3] show alias definitions in help
Message-ID: <alpine.LNX.1.00.0802251136190.19024@iabervon.org>
URL: https://gitlist.dev/e/alpine.LNX.1.00.0802251136190.19024%40iabervon.org
In-Reply-To: <20080224222331.GA6712@coredump.intra.peff.net>

```

On Sun, 24 Feb 2008, Jeff King wrote:

> BTW, Daniel, I have used your --cover-letter work several times today,
> and it is terrific. It has saved me a lot of error-prone cutting and
> pasting of message-ids and commit titles. Thanks.

Great to hear. Let me know if you have any suggestions to improve it; I 
haven't been using it all that much, since I got it into my installed git 
approximately when I'd just cleared my table of long series, so you may 
come up with obvious lacks before I do.

	-Daniel
*This .sig left intentionally blank*

```