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

Re: [PATCHv2] git-p4: clean up after p4 submit failure

From
Jeff King <peff@peff.net>
Date
Nov 24, 2015, 21:18 UTC
Message-ID
<20151124211827.GA29185@sigill.intra.peff.net>
In-Reply-To
<1448351039-15625-2-git-send-email-luke@diamand.org>
On Tue, Nov 24, 2015 at 07:43:59AM +0000, Luke Diamand wrote:
Show 15 quoted lines
> From: GIRARD Etienne <egirard@murex.com>
> 
> When "p4 submit" command fails in P4Submit.applyCommit, the
> workspace is left with the changes.  We already have code to revert
> the changes to the workspace when the user decides to cancel
> submission by aborting the editor that edits the change description,
> and we should treat the "p4 submit" failure the same way.
> 
> Clean the workspace if p4_write_pipe raised SystemExit, so that the
> user don't have to do it themselves.
> 
> Signed-off-by: GIRARD Etienne <egirard@murex.com>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> Signed-off-by: Luke Diamand <luke@diamand.org>
> ---
Thanks, I think this should be ready for 'next', then.
-Peff
Previous: Luke Diamand
Message 3 of 3 in “[PATCHv2] git-p4: Handle p4 submit failure”
  1. Luke DiamandNov 24, 2015
  2. [PATCHv2] git-p4: clean up after p4 submit failureLuke Diamand, Nov 24, 2015
  3. Jeff KingNov 24, 2015

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.