Junio C Hamano wrote: > I'll queue these for now probably on 'pu', but with the comments we saw on > the list expect them to be followed up with replacement patches. Here’s a replacement series. It omits the longer error message when TERM=dumb and the git var --run experiment because I was not happy with where either of those were going. Thanks for all the comments, everyone. Johannes Sixt (1): Teach git var about GIT_EDITOR Jonathan Nieder (6): Handle more shell metacharacters in editor names Do not use VISUAL editor on dumb terminals Teach git var about GIT_PAGER add -i, send-email, svn, p4, etc: use "git var GIT_EDITOR" am -i, git-svn: use "git var GIT_PAGER" Provide a build time default-editor setting Junio C Hamano (1): Provide a build time default-pager setting Documentation/config.txt | 4 +--- Documentation/git-commit.txt | 2 +- Documentation/git-send-email.txt | 4 ++-- Documentation/git-var.txt | 14 ++++++++++++++ Makefile | 28 ++++++++++++++++++++++++++++ cache.h | 2 ++ contrib/fast-import/git-p4 | 5 +---- editor.c | 32 +++++++++++++++++++++++--------- git-add--interactive.perl | 3 +-- git-am.sh | 5 ++++- git-send-email.perl | 3 ++- git-sh-setup.sh | 19 ++++++------------- git-svn.perl | 11 ++++------- pager.c | 24 ++++++++++++++++++++---- t/t7005-editor.sh | 31 ++++++++++++++++++++++++------- var.c | 31 ++++++++++++++++++++++++++++++- 16 files changed, 163 insertions(+), 55 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html