Handle all hostaliases in /etc/hosts

[akdevservices]

Fixes #3488

[junegunn]

Thanks, it looks like we should handle in-line comments.

127.0.0.1	localhost home # homey

What do you think?

diff --git a/shell/completion.bash b/shell/completion.bash
index c42909b..db84e06 100644
--- a/shell/completion.bash
+++ b/shell/completion.bash
@@ -423,7 +423,7 @@ if ! declare -F __fzf_list_hosts > /dev/null; then
   __fzf_list_hosts() {
     command cat <(command tail -n +1 ~/.ssh/config ~/.ssh/config.d/* /etc/ssh/ssh_config 2> /dev/null | command grep -i '^\s*host\(name\)\? ' | command awk '{for (i = 2; i <= NF; i++) print $1 " " $i}' | command grep -v '[*?%]') \
       <(command grep -oE '^[[a-z0-9.,:-]+' ~/.ssh/known_hosts 2> /dev/null | command tr ',' '\n' | command tr -d '[' | command awk '{ print $1 " " $1 }') \
-      <(command grep -v '^\s*\(#\|$\)' /etc/hosts 2> /dev/null | command grep -Fv '0.0.0.0') |
+      <(command grep -v '^\s*\(#\|$\)' /etc/hosts 2> /dev/null | command grep -Fv '0.0.0.0' | command sed 's/#.*//') |
       command awk '{for (i = 2; i <= NF; i++) print $i}' | command sort -u
   }
 fi
diff --git a/shell/completion.zsh b/shell/completion.zsh
index 1189c26..c4ba2c7 100644
--- a/shell/completion.zsh
+++ b/shell/completion.zsh
@@ -226,7 +226,7 @@ if ! declare -f __fzf_list_hosts > /dev/null; then
     setopt localoptions nonomatch
     command cat <(command tail -n +1 ~/.ssh/config ~/.ssh/config.d/* /etc/ssh/ssh_config 2> /dev/null | command grep -i '^\s*host\(name\)\? ' | awk '{for (i = 2; i <= NF; i++) print $1 " " $i}' | command grep -v '[*?%]') \
       <(command grep -oE '^[[a-z0-9.,:-]+' ~/.ssh/known_hosts 2> /dev/null | tr ',' '\n' | tr -d '[' | awk '{ print $1 " " $1 }') \
-      <(command grep -v '^\s*\(#\|$\)' /etc/hosts 2> /dev/null | command grep -Fv '0.0.0.0') |
+      <(command grep -v '^\s*\(#\|$\)' /etc/hosts 2> /dev/null | command grep -Fv '0.0.0.0' | command sed 's/#.*//') |
       awk '{for (i = 2; i <= NF; i++) print $i}' | sort -u
   }
 fi

[akdevservices]

What do you think?

It's a fix for another issue :) I've included it in this pull request.

[junegunn]

Merged, thanks!