-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Description
I have installed Git Extensions 2.32 onto a Windows XP SP3 32-bit machine and found out that simply running git commit
(from cmd.exe) in a repository suddenly spawns some unexpected test editor.
As it turned out, this is a text editor provided by Git Extensions:
C:>git config --get-all core.editor
"C:/Program Files/GitExtensions/GitExtensions.exe" fileeditor
Note that I do not have core.editor set at all (on either condiguration level) and instead relying on the EDITOR environment variable being set to "gvim.exe" (as thisway to set the preferred editor is used by a number of other tools, like command-line subversion client).
I do not remember Git Extensions asked me about this commit editor setting during the setup phase nor did I change anything relevant in its settings dialog (I only messed with the language and PuTTY location).
So it looks like either Git Extensions unconditionally hijacks this setting unconditionally (I doubt this) or relies or somewhat naive heuristics ("the user does not have core.editor set so we're free to set it to whatever we wish").
I think the proper way to go would be to ask the user on install or, if heuristics are used, to at least look at the environment and do nothing if either VISUAL or EDITOR is set.