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

Re: [PATCH] git-submodule: Make update verbose for up to date path

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Mar 4, 2008, 16:08 UTC
Message-ID
<alpine.LSU.1.00.0803041605450.22527@racer.site>
In-Reply-To
<46dff0320803040803v2f93d749t6b45fc8e47e66d21@mail.gmail.com>
Hi,
On Wed, 5 Mar 2008, Ping Yin wrote:
> When i run 'git submodule update' in a repository whose submodules are 
> all clean, there is not any output which makes me not know whether this 
> command are successful. I think this is not friendly.

The problem is: people ignore output. Back when merge-recursive was even more chatty than it is now, they happily glanced over the fact that there were _conflicts_.

So we reduced the output.  Drastically.

For example, git reset --hard (amongst others) does not output a _thing_ if it succeeds. This is good. No output means it succeeded.

Now, some other commands take a while to complete, so we have a progress which kicks in after a second or so. Still, this is just to keep the user busy when she would otherwise think that Git hangs.

So no, I do not like submodule becoming chattier. You can add --verbose if you want, but the default should be _quiet_.

Ciao, Dscho

Previous: Ping YinNext: Junio C Hamano
Message 4 of 5 in “git-submodule: Make update verbose for up to date path”
  1. git-submodule: Make update verbose for up to date pathPing Yin, Mar 4, 2008
  2. Johannes SchindelinMar 4, 2008
  3. Ping YinMar 4, 2008
  4. Johannes SchindelinMar 4, 2008
  5. Junio C HamanoMar 4, 2008

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.