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

Re: [PATCH] rebase -i: clean error message for --continue after failed exec

From
Jeff King <peff@peff.net>
Date
Aug 24, 2011, 20:20 UTC
Message-ID
<20110824202027.GA28900@sigill.intra.peff.net>
In-Reply-To
<7v62lmps6k.fsf@alter.siamese.dyndns.org>
On Wed, Aug 24, 2011 at 11:42:27AM -0700, Junio C Hamano wrote:
Show 6 quoted lines
> Matthieu Moy <Matthieu.Moy@imag.fr> writes:
> 
> > If after a failed "exec" instruction there are staged changes,...
> 
> I have to wonder why whatever "exec" runs is mucking with the index in the
> first place. Shouldn't we forbid it?

Certainly my only user has ever been "exec make test". But I wonder if somebody is crazy enough to auto-generate some content and commit it. OTOH, shouldn't it then be their responsibility to make the commit? I.e., I can see at least the potential for mucking with the index, but I really don't see a reason for _leaving_ the index in a mucked state.

-Peff
Previous: Johannes SixtNext: Jeff King
Message 8 of 9 in “rebase -i: Should --continue auto-amend after failed exec?”
  1. Johannes SixtAug 8, 2011
  2. Matthieu MoyAug 24, 2011
  3. rebase -i: clean error message for --continue after failed execMatthieu Moy, Aug 24, 2011
  4. Junio C HamanoAug 24, 2011
  5. Junio C HamanoAug 24, 2011
  6. Matthieu MoyAug 25, 2011
  7. Johannes SixtAug 25, 2011
  8. Jeff KingAug 24, 2011
  9. Jeff KingAug 24, 2011

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.