Thread (3 messages) flat view 3 messages, 3 authors, 2016-06-15

Re: [PATCH] rebase -i: fix reword when using a terminal editor

From: Björn Gustavsson <hidden>
Date: 2016-06-15 22:47:34

2009/10/16 Stephen Boyd [off-list ref]:
We don't want to use output() on git-commit --amend when rewording the
commit message. This leads to confusion as the editor is run in a
subshell with it's output saved away, leaving the user with a seemingly
frozen terminal.

Fix by removing the output part.

Signed-off-by: Stephen Boyd <redacted>
---

On top of next.

I think this is right, although I'm not really experienced in the rebase -i
code
Thanks!

Yes, it seems right to me too after a more thorough reading of the
existing rebase -i code. It seems that 'output' is only used for
non-interactive commands.

I never noticed the problem because I use 'emacsclient' as my
commit editor. I didn't think of testing with an editor that run in
the terminal window.

/Björn
-- 
Björn Gustavsson, Erlang/OTP, Ericsson AB
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help