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

Re: [PATCH 0/2] log/ format-patch improvements

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Aug 25, 2010, 20:54 UTC
Message-ID
<20100825205416.GB2319@burratino>
In-Reply-To
<20100825084416.GC3280@kytes>
Ramkumar Ramachandra wrote:
> Ramkumar Ramachandra writes:
>> The second patch clarifies the meaning of the `-<n>` option. We should
>> also probably force the mutual exclusivity of `-<n>` and <revision
>> range> to avoid confusion.
[...]
> Do you see value in this patch or is it just unnecessary baggage?

I see value in avoiding confusion. Maybe one solution would be to make format-patch use --no-merges by default.

 $ git log --oneline --no-merges -3 ab/test..origin/pu
 70256a3 shell: Rewrite documentation and improve error message
 9c46c05 rev-parse: tests git rev-parse --verify master@{n}, for various n
 eedce78 sha1_name.c: use warning in preference to fprintf(stderr
Previous: Ramkumar RamachandraNext: Ramkumar Ramachandra
Message 5 of 11 in “log/ format-patch improvements”
  1. 0/2 log/ format-patch improvementsRamkumar Ramachandra, Aug 21, 2010
  2. 1/2 git-format-patch: Print a diagnostic message when ignoring commitsRamkumar Ramachandra, Aug 21, 2010
  3. 2/2 log: Improve description of '-<n>' option in documentationRamkumar Ramachandra, Aug 21, 2010
  4. Ramkumar RamachandraAug 25, 2010
  5. Jonathan NiederAug 25, 2010
  6. Ramkumar RamachandraAug 26, 2010
  7. Jonathan NiederAug 26, 2010
  8. Thomas RastAug 26, 2010
  9. Junio C HamanoAug 26, 2010
  10. Ramkumar RamachandraAug 26, 2010
  11. Junio C HamanoAug 25, 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.