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

Re: [PATCH] status&commit: Teach them to show submodule commit summary

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Nov 12, 2007, 16:13 UTC
Message-ID
<Pine.LNX.4.64.0711121612250.4362@racer.site>
In-Reply-To
<4738784F.7010106@viscovery.net>
Hi,
On Mon, 12 Nov 2007, Johannes Sixt wrote:
Show 17 quoted lines
> Ping Yin schrieb:
> > git status/commit just treats submodules as ordinary files when 
> > reporting status changes. However, one may also wonder how submodules 
> > change (the commits).
> > 
> > This commit teaches git status/commit to additionally show commit 
> > summary of user-cared (i.e. checked out) modified submodules since 
> > HEAD (or HEAD^ if --amend option is on). For submodules deleted or 
> > initially added, commit summary are not shown.
> 
> In general, I like the idea (as I've already pointed out).
> 
> But at this time git-commit is about to be made a builtin, and since 
> your implementation contains a lot of non-portable constructs ($'', >&) 
> and a new dependency on awk (and, hence, has little chances of being 
> accepted), I suggest that you stay tuned, and implement this in the 
> forth-coming builtin-commit.c.
I agree.
Show 6 quoted lines
> > A configuration variable 'submodule.status' is used to turn this 
> > summary behaviour on or off (default off). Also --submodule and 
> > --no-submodule options are added.
> 
> There is already 'status.color', I suggest the configuration to become 
> 'status.submoduleSummary'.

You could make this variable even a numeric one. Saying how many lines you want to have maximally (-1 meaning unlimited).

Ciao, Dscho

Previous: Ping YinNext: Ping Yin
Message 10 of 16 in “status&commit: Teach them to show submodule commit summary”
  1. status&commit: Teach them to show submodule commit summaryPing Yin, Nov 12, 2007
  2. Ralf WildenhuesNov 12, 2007
  3. Ping YinNov 12, 2007
  4. Brian GernhardtNov 12, 2007
  5. Jakub NarebskiNov 12, 2007
  6. Ping YinNov 12, 2007
  7. Johannes SixtNov 12, 2007
  8. Jakub NarebskiNov 12, 2007
  9. Ping YinNov 12, 2007
  10. Johannes SchindelinNov 12, 2007
  11. Ping YinNov 12, 2007
  12. Johannes SixtNov 12, 2007
  13. Ping YinNov 12, 2007
  14. Johannes SixtNov 12, 2007
  15. Lars HjemliNov 12, 2007
  16. Ping YinNov 15, 2007

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.