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

Re: [RFC][Feature] submodule: teach subcommand 'summary' to give summary for nested submodules

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 18, 2020, 19:48 UTC
Message-ID
<xmqqh7ylfnqi.fsf@gitster.c.googlers.com>
In-Reply-To
<20200318163234.21628-1-shouryashukla.oo@gmail.com>
Shourya Shukla <shouryashukla.oo@gmail.com> writes:
Show 9 quoted lines
> I was converting the subcommand 'summary'(of command submodule)
> from shell to C and I realised that the subcommand does not have a
> '--recursive' option in it, meaning it cannot give out summaries
> of nested submodules. My opinion on this is that an option should
> maybe exist for the same.
>
> What are the community's thoughts on this? Should the
> aforementioned option be added? Is there any feasibility of this
> option in this case?

If "git submodule summary --recursive" errors out with today's code, no sane user would be using it for any useful purpose, so I would think it is OK to add such a feature.

If it makes a usable UI for large projects that you can only choose between "include no subsubmodules" (i.e. with --no-recursive) or "include all sub(sub)*modules" (i.e. with --recursive) is another matter, though.

Previous: Shourya ShuklaNext: Shourya Shukla
Message 2 of 6 in “RFC][Feature] submodule: teach subcommand 'summary' to give summary for nested submodules”
  1. Shourya ShuklaMar 18, 2020
  2. Junio C HamanoMar 18, 2020
  3. Shourya ShuklaMar 19, 2020
  4. Junio C HamanoMar 19, 2020
  5. Shourya ShuklaMar 20, 2020
  6. Christian CouderMar 24, 2020

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.