Add abstraction to check for GUI callback support

Author: tpope

autoload/dispatch.vim

@@ -172,6 +172,10 @@ function! dispatch#vim_executable() abort
   return s:vim
 endfunction
 
+function! dispatch#has_callback() abort
+  return has('clientserver') && !empty(v:servername)
+endfunction
+
 function! dispatch#callback(request) abort
   if has('clientserver') && !empty(v:servername) && has_key(s:request(a:request), 'id')
     return dispatch#shellescape(dispatch#vim_executable()) .

autoload/dispatch/iterm.vim

@@ -10,7 +10,7 @@ function! dispatch#iterm#handle(request) abort
     return 0
   endif
   if a:request.action ==# 'make'
-    if !get(a:request, 'background', 0) && !has('gui_running')
+    if !get(a:request, 'background', 0) && !dispatch#has_callback()
       return 0
     endif
     let exec = dispatch#prepare_make(a:request)

autoload/dispatch/screen.vim

@@ -10,7 +10,7 @@ function! dispatch#screen#handle(request) abort
     return 0
   endif
   if a:request.action ==# 'make'
-    if !get(a:request, 'background', 0) && empty(v:servername)
+    if !get(a:request, 'background', 0) && !dispatch#has_callback()
       return 0
     endif
     return dispatch#screen#spawn(dispatch#prepare_make(a:request), a:request)

autoload/dispatch/tmux.vim

@@ -18,7 +18,7 @@ function! dispatch#tmux#handle(request) abort
   endif
 
   if a:request.action ==# 'make'
-    if !get(a:request, 'background', 0) && empty(v:servername) &&
+    if !get(a:request, 'background', 0) && !dispatch#has_callback() &&
           \ !empty(''.session) && session !=# system('tmux display-message -p "#S"')[0:-2]
       return 0
     endif

autoload/dispatch/x11.vim

@@ -23,7 +23,7 @@ function! dispatch#x11#handle(request) abort
     return 0
   endif
   if a:request.action ==# 'make'
-    if !get(a:request, 'background', 0) && empty(v:servername)
+    if !get(a:request, 'background', 0) && !dispatch#has_callback()
       return 0
     endif
     return dispatch#x11#spawn(terminal, dispatch#prepare_make(a:request), a:request)