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

Re: Bug? diff.submodule=log adds text to commit -v message

From
Jens Lehmann <jens.lehmann@web.de>
Date
Nov 11, 2013, 20:41 UTC
Message-ID
<528140F5.6090700@web.de>
In-Reply-To
<loom.20131110T222043-57@post.gmane.org>
Hi Ari,
Am 10.11.2013 22:49, schrieb Ari Pollak:
Show 6 quoted lines
> I'm using git 1.8.4.2, and I've set the "diff.submodule = log" option 
> globally. If I change the revision that a submodule is set to, then run
> "git commit -av", The submodule shortlog is appended to the log message without 
> any #s before it, so the log messages get included in my own log message. 
> This seems like a bug and not a feature, as diffs aren't normally included in 
> the commit message with -v.

Thanks for your report, I can reproduce that here. But first I think this is unrelated to the "diff.submodule = log" setting, as without it you'll just see the submodule commit hash diff instead of the shortlog (which is perfectly consistent with what I'd expect from this setting). And secondly what you describe looks like documented behavior, the man page of "git commit" states:

 -v, --verbose
     Show unified diff between the HEAD commit and what would be
     committed at the bottom of the commit message template. Note that
     this diff output doesn't have its lines prefixed with #.

And after adding a modified file the log message also shows the diff of that file (and without leading "# "s too), so I doubt that diffs aren't normally included in the commit message with -v. What am I missing?

Thanks Jens

Previous: Ari PollakNext: Ari Pollak
Message 2 of 23 in “Bug? diff.submodule=log adds text to commit -v message”
  1. Ari PollakNov 10, 2013
  2. Jens LehmannNov 11, 2013
  3. Ari PollakNov 11, 2013
  4. Jens LehmannNov 11, 2013
  5. Jens LehmannNov 11, 2013
  6. Jeff KingNov 11, 2013
  7. Johannes SixtNov 12, 2013
  8. Jens LehmannNov 12, 2013
  9. Junio C HamanoNov 12, 2013
  10. Jens LehmannNov 13, 2013
  11. Junio C HamanoNov 13, 2013
  12. commit -v: strip diffs and submodule shortlogs from the commit messageJens Lehmann, Nov 16, 2013
  13. Eric SunshineNov 17, 2013
  14. Jeff KingNov 17, 2013
  15. Jeff KingNov 17, 2013
  16. Jens LehmannNov 17, 2013
  17. Junio C HamanoNov 18, 2013
  18. commit -v: strip diffs and submodule shortlogs from the commit messageJens Lehmann, Nov 19, 2013
  19. Junio C HamanoNov 19, 2013
  20. Junio C HamanoNov 19, 2013
  21. Junio C HamanoNov 19, 2013
  22. Junio C HamanoNov 19, 2013
  23. Jens LehmannNov 19, 2013

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.