From 9cf20dbff5fad338c48f6509dc1f4b452639686f Mon Sep 17 00:00:00 2001
From: Tim Pope <code@tpope.net>
Date: Fri, 20 Aug 2021 08:57:03 -0400
Subject: [PATCH] Don't set $COLUMNS on legacy :! implementation

It's unnecessary and offensive to the eye.
---
 autoload/fugitive.vim | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/autoload/fugitive.vim b/autoload/fugitive.vim
index 5804ada..5465d0f 100644
--- a/autoload/fugitive.vim
+++ b/autoload/fugitive.vim
@@ -3393,7 +3393,6 @@ function! fugitive#Command(line1, line2, range, bang, mods, arg) abort
     let pager = 1
     let stream = exists('*setbufline')
     let do_edit = substitute(s:Mods(a:mods, &splitbelow ? 'botright' : 'topleft'), '\<tab\>', '-tab', 'g') . 'pedit!'
-    call extend(env, {'COLUMNS': '' . (&columns - 1)}, 'keep')
   elseif pager
     let allow_pty = 0
     if pager is# 2 && a:bang && a:line2 >= 0
@@ -3407,10 +3406,9 @@ function! fugitive#Command(line1, line2, range, bang, mods, arg) abort
       call s:BlurStatus()
     endif
     call extend(env, {'COLUMNS': '' . get(g:, 'fugitive_columns', 80)}, 'keep')
-  else
-    call extend(env, {'COLUMNS': '' . (&columns - 1)}, 'keep')
   endif
   if s:RunJobs()
+    call extend(env, {'COLUMNS': '' . (&columns - 1)}, 'keep')
     let state.pty = allow_pty && get(g:, 'fugitive_pty', has('unix') && !has('win32unix') && (has('patch-8.0.0744') || has('nvim')) && fugitive#GitVersion() !~# '\.windows\>')
     if !state.pty
       let args = s:AskPassArgs(dir) + args