Re: [PATCH v3] launch_editor(): indicate that Git waits for user input
- From
brian m. carlson <sandals@crustytoothpaste.net>
- Date
- Nov 27, 2017, 20:09 UTC
- Message-ID
- <20171127200932.GA558049@genre.crustytoothpaste.net>
- In-Reply-To
- <20171127134716.69471-1-lars.schneider@autodesk.com>
On Mon, Nov 27, 2017 at 02:47:16PM +0100, lars.schneider@autodesk.com wrote:
Show 22 quoted lines
> From: Junio C Hamano <gitster@pobox.com>
>
> When a graphical GIT_EDITOR is spawned by a Git command that opens
> and waits for user input (e.g. "git rebase -i"), then the editor window
> might be obscured by other windows. The user may be left staring at the
> original Git terminal window without even realizing that s/he needs to
> interact with another window before Git can proceed. To this user Git
> appears hanging.
>
> Show a message in the original terminal and get rid of it when the
> editor returns.
> diff --git a/editor.c b/editor.c
> index 7519edecdc..4251ae9d7a 100644
> --- a/editor.c
> +++ b/editor.c
> @@ -40,6 +40,35 @@ int launch_editor(const char *path, struct strbuf *buffer, const char *const *en
> const char *args[] = { editor, real_path(path), NULL };
> struct child_process p = CHILD_PROCESS_INIT;
> int ret, sig;
> + static const char *close_notice = NULL;
> +
> + if (isatty(2) && !close_notice) {Sorry for coming to the topic so late, but it occurred to me that we might want to conditionalize this on an advice.* flag. I expect there are some people who will never want to see this, and letting them turn it off would be good.
-- brian m. carlson / brian with sandals: Houston, Texas, US https://www.crustytoothpaste.net/~bmc | My opinion only OpenPGP: https://keybase.io/bk2204