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

Re: [PATCH] git-submodule: Don't die when command fails for one submodule

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 4, 2008, 20:32 UTC
Message-ID
<7vzlte44dg.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<46dff0320803040806y5ed02b86p9216ddc752f3052@mail.gmail.com>
"Ping Yin" <pkufranky@gmail.com> writes:
Show 15 quoted lines
> On Wed, Mar 5, 2008 at 12:03 AM, Johannes Schindelin
> <Johannes.Schindelin@gmx.de> wrote:
>> Hi,
>>
>>
>>  On Wed, 5 Mar 2008, Ping Yin wrote:
>>
>>  > The error output is not lost and is in module_name
>>
>>  That's what I am saying: the error output is not enough.  The exit status
>>  needs to indicate that there was an error, too.  Everything else is
>>  unusable by scripts.
>
> I think this can be done in a following patch which capture any error
> and give the right exit status.
Please.

"You are absolutely right. I'll fix it in my next round to do frotz and xyzzy as you suggested. I want to make my next round better with fixes and improvements based on other comments I will receive during this round, so it will be at least in a few days. Thanks."

That's how I want to see things work around here.
Previous: Johannes Schindelin
Message 7 of 7 in “git-submodule: Don't die when command fails for one submodule”
  1. git-submodule: Don't die when command fails for one submodulePing Yin, Mar 4, 2008
  2. Johannes SchindelinMar 4, 2008
  3. Ping YinMar 4, 2008
  4. Johannes SchindelinMar 4, 2008
  5. Ping YinMar 4, 2008
  6. Johannes SchindelinMar 4, 2008
  7. 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.