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

Re: [GSoC][PATCH 1/1] sequencer: print an error message if append_todo_help() fails

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jun 26, 2018, 12:57 UTC
Message-ID
<nycvar.QRO.7.76.6.1806261455510.21419@tvgsbejvaqbjf.bet>
In-Reply-To
<20180626100429.10169-2-alban.gruin@gmail.com>
Hi Alban,
On Tue, 26 Jun 2018, Alban Gruin wrote:
> This adds an error when append_todo_help() fails to write its message to
> the todo file.
> 
> Signed-off-by: Alban Gruin <alban.gruin@gmail.com>
ACK.

We *may* want to fold that into the commit that adds `append_todo_help()`. And, as I mentioned previously, I would love for that function to be used as an excuse to introduce the long-overdue `interactive-rebase.c` (`sequencer.c` is supposed to be the backend for cherry-pick and revert and interactive rebase, but not a catch-all for *all* of those things, it is already way too long to be readable, and I take blame for a large part of that.)

Ciao, Dscho

Previous: Alban GruinNext: Junio C Hamano
Message 3 of 5 in “sequencer: print an error message if append_todo_help() fails”
  1. Alban GruinJun 26, 2018
  2. [GSoC][PATCH 1/1] sequencer: print an error message if append_todo_help() failsAlban Gruin, Jun 26, 2018
  3. Johannes SchindelinJun 26, 2018
  4. Junio C HamanoJun 26, 2018
  5. Johannes SchindelinJun 26, 2018

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.