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

Re: [PATCH] [RFC] Add a new config. option for skipping merges in git-log

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 16, 2015, 20:50 UTC
Message-ID
<xmqqy4mwpsyn.fsf@gitster.dls.corp.google.com>
In-Reply-To
<5507301B.2060401@posteo.de>
Koosha Khajehmoogahi <koosha@posteo.de> writes:
Show 5 quoted lines
> Thanks for your suggestions. The "extra bit" in rev_info is used when
> we need to compare user's command-line input to his configuration. Since
> command-line input is processed in revision.c but config. options are read
> in builtin/log.c, we need a way to pass the value to builtin/log.c. How
> would you do that without this extra bit?

The command line overrides the config, no? If you set up what the command line defaults to from the config, let the command line parser do whatever it wants to do, and do nothing else after the command line parser returns, wouldn't that be sufficient?

Puzzled...
Previous: Koosha KhajehmoogahiNext: Koosha Khajehmoogahi
Message 6 of 8 in “[RFC] Add a new config. option for skipping merges in git-log”
  1. [RFC] Add a new config. option for skipping merges in git-logKoosha Khajehmoogahi, Mar 16, 2015
  2. [RFC] Add a new config. option for skipping merges in git-logKoosha Khajehmoogahi, Mar 16, 2015
  3. [RFC] Add a new config. option for skipping merges in git-logKoosha Khajehmoogahi, Mar 16, 2015
  4. Junio C HamanoMar 16, 2015
  5. Koosha KhajehmoogahiMar 16, 2015
  6. Junio C HamanoMar 16, 2015
  7. Koosha KhajehmoogahiMar 16, 2015
  8. Junio C HamanoMar 16, 2015

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.