From 63310f6403474e8b5c586627edf97b16c6e41aa4 Mon Sep 17 00:00:00 2001
From: Kaleb Elwert <belak@coded.io>
Date: Wed, 4 Oct 2017 11:27:55 -0700
Subject: [PATCH] ssh: update SSH_ASKPASS workaround to more closely match what
 ssh-add needs

Fixes #1478
---
 modules/ssh/init.zsh | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/modules/ssh/init.zsh b/modules/ssh/init.zsh
index c0316d2..6e37b69 100644
--- a/modules/ssh/init.zsh
+++ b/modules/ssh/init.zsh
@@ -39,11 +39,20 @@ fi
 # Load identities.
 if ssh-add -l 2>&1 | grep -q 'The agent has no identities'; then
   zstyle -a ':prezto:module:ssh:load' identities '_ssh_identities'
-  # Check for Linux system and ssh-askpass presence
-  if [[ "$OSTYPE" == linux* ]] && [[ ! -a /usr/lib/ssh/x11-ssh-askpass ]]; then
-    ssh-add "${_ssh_identities:+$_ssh_dir/${^_ssh_identities[@]}}"  2> /dev/null
+  # ssh-add has strange requirements for running SSH_ASKPASS, so we duplicate
+  # them here. Essentially, if the other requirements are met, we redirect stdin
+  # from /dev/null in order to meet the final requirement.
+  #
+  # From ssh-add(1):
+  # If ssh-add needs a passphrase, it will read the passphrase from the current
+  # terminal if it was run from a terminal. If ssh-add does not have a terminal
+  # associated with it but DISPLAY and SSH_ASKPASS are set, it will execute the
+  # program specified by SSH_ASKPASS and open an X11 window to read the
+  # passphrase.
+  if [[ -n "$DISPLAY" && -x "$SSH_ASKPASS" ]]; then
+    ssh-add "${_ssh_identities:+$_ssh_dir/${^_ssh_identities[@]}}" < /dev/null 2> /dev/null
   else
-    ssh-add "${_ssh_identities:+$_ssh_dir/${^_ssh_identities[@]}}"  < /dev/null 2> /dev/null
+    ssh-add "${_ssh_identities:+$_ssh_dir/${^_ssh_identities[@]}}" 2> /dev/null
   fi
 fi