Re: [PATCH] git-send-email: ssh/login style password requests

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH] git-send-email: ssh/login style password requests

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:09

Michael Witten [off-list ref] writes:
It turns out that --compose will also leave some temporary files
behind if git-send-email is terminated out of course.

Should I establish a general handler for SIGTERM and SIGINT?
I was mostly worried about ^C leaving the terminal in a funny
state with your system(stty).

I do not think you should lose the message the user carefully
composed using an editor upon seeing ^C.  The user may start to
compose, edit the message, and then at some later time decide to
stop the mail-sending procedure by ^C-ing out.  The user would
thank you if you somehow left an escape hatch to allow salvaging
the message, instead of having it to be re-typed from scratch.

Re: [PATCH] git-send-email: ssh/login style password requests

From: Michael Witten <hidden>
Date: 2016-06-15 22:44:09

On 1 Feb 2008, at 6:34 PM, Junio C Hamano wrote:
Michael Witten [off-list ref] writes:
quoted
It turns out that --compose will also leave some temporary files
behind if git-send-email is terminated out of course.

Should I establish a general handler for SIGTERM and SIGINT?
I was mostly worried about ^C leaving the terminal in a funny
state with your system(stty).

I do not think you should lose the message the user carefully
composed using an editor upon seeing ^C.  The user may start to
compose, edit the message, and then at some later time decide to
stop the mail-sending procedure by ^C-ing out.  The user would
thank you if you somehow left an escape hatch to allow salvaging
the message, instead of having it to be re-typed from scratch.
Indeed. How about a message stating that these files exist?

In any case, I'm looking for your endorsement of a signal handler
as the means to solve the main problem.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help