git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/3] show alias definitions in help

From
Jeff King <peff@peff.net>
Date
Feb 24, 2008, 22:16 UTC
Message-ID
<20080224221559.GA31309@coredump.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
Next: Jeff King
Message 1 of 3 in “show alias definitions in help”
  1. 0/3 show alias definitions in helpJeff King, Feb 24, 2008
  2. Jeff KingFeb 24, 2008
  3. Daniel BarkalowFeb 25, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.