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

[PATCH v4 0/3] pretty: format aliases

From
Will Palmer <wmpalmer@gmail.com>
Date
May 2, 2010, 11:00 UTC
Message-ID
<1272798044-10487-1-git-send-email-wmpalmer@gmail.com>

The following patch series adds the ability to configure aliases for user-defined formats. The first two patches add infrastructure for the ease of adding additional formats, and infrastructure for defining format aliases, respectively. The final patch adds support for defining format aliases via a config option.

This is the fourth version of the patch series. The most notable change from the last iteration is the removal of changes to other pretty-format options, such as the "conditional colors" patch and "%H/%h --abbrev" patches. I still think those were good patches, but the %H change in particular was controversial enough that it would have needlessly stood in the way of the rest of the series, and '%?C' looked out-of-place in this series without any other format changes. The removed patches will be worked on independently and will be submitted separately later.

Jeff King noted an artifact of format.pretty.<name> left in the documentation, thanks.

Junio asked for a better error message when looped-aliases were found, so that's been put in.

The copyright notice for the added test has been changed to the standard boilerplate, because there's no reason to bring politics into a patch.

Most other changes since the last iteration were simple style fixes, mostly a result of muscle memory from the coding conventions I use at work, though a couple of things (like not initializing static variables) were more towards the "you can do that? But my CS teacher in highschool told me not to!" end of the spectrum. Hopefully I've caught all the style problems this time.

Will Palmer (3):
  pretty: make it easier to add new formats
  pretty: add infrastructure to allow format aliases
  pretty: add aliases for pretty formats
 Documentation/config.txt         |    9 ++
 Documentation/pretty-formats.txt |    7 ++-
 pretty.c                         |  154 ++++++++++++++++++++++++++++++++------
 t/t4205-log-pretty-formats.sh    |   66 ++++++++++++++++
 4 files changed, 212 insertions(+), 24 deletions(-)
 create mode 100755 t/t4205-log-pretty-formats.sh
Next: Will Palmer
Message 1 of 11 in “pretty: format aliases”
  1. 0/3 pretty: format aliasesWill Palmer, May 2, 2010
  2. 1/3 pretty: make it easier to add new formatsWill Palmer, May 2, 2010
  3. Jonathan NiederMay 2, 2010
  4. 2/3 pretty: add infrastructure to allow format aliasesWill Palmer, May 2, 2010
  5. Jonathan NiederMay 2, 2010
  6. 3/3 pretty: add aliases for pretty formatsWill Palmer, May 2, 2010
  7. Jonathan NiederMay 2, 2010
  8. Will PalmerMay 2, 2010
  9. pretty: initialize new cmt_fmt_map to 0Jonathan Nieder, May 8, 2010
  10. Will PalmerMay 8, 2010
  11. Junio C HamanoMay 2, 2010

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.