Re: [PATCH 6/6] diff-merges: let -m imply -p

From: Jonathan Nieder <jrnieder@gmail.com>
Date: May 19, 2021, 21:48 UTC
Message-ID: <YKWHo9gZNp+i3fH9@google.com>
In-Reply-To: <20210510153451.15090-7-sorganov@gmail.com>

Sergey Organov wrote:

Show 8 quoted lines

> Fix long standing inconsistency between -c/--cc that do imply -p, on
> one side, and -m that did not imply -p, on the other side.
>
> After this patch
>
>   git log -m
>
> will start to produce diffs without need to provide -p as well,

Personally I don't ever use -m without -p and --first-parent, so in that sense this feels like a change in the right direction.

Does this also affect the plumbing command "git diff-tree"? I'm guessing "no" because diff-tree already generates a diff by default, but it seems worth spelling out in the commit message to prevent worries about the effect on scripts that expect stable plumbing behavior.

Thanks, Jonathan

$ 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.