On Sun, Nov 20, 2022 at 12:53 AM Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > Would it make sense to expand this to support the VISUAL and EDITOR > environment variables too? For instance, since I want to use Emacs for > all my editing needs, I have EDITOR set to reference emacsclient. So, > if GIT_EDITOR is unset, then check VISUAL, and if that is unset, check > EDITOR. "git var GIT_EDITOR" does respect VISUAL and EDITOR environment variables. See git-var(1) reference manual. > Also, on macOS, I need to set EDITOR (or GIT_EDITOR or VISUAL) to the > full path of emacsclient, but the regex used here is too tight to > recognize that. Perhaps loosening it to just 'emacsclient' would be a > good idea (as it seems unlikely to falsely match any other editor)? It is a good idea. Thanks! -- Yoichi NAKAYAMA
