08da19e3fd27acf5085e207ee7da411e6a5eb96a

Author
akinnane <akinnane@yahoo.co.uk>
Committer
Marc Cornellà <marc.cornella@live.com>
Date

Message

emacs: revert cab1ac6e682038b2ceca9d16f8c458c0e79149ca (#7765)

Revert cab1ac6e682038b2ceca9d16f8c458c0e79149ca

The change from the above commit breaks opening a new emacs frame if
there isn't a existing frame open.

When emacs is running in daemon mode, there will always be a frame
associated with the daemon, even if there are no visible frames.

The lisp function `frame-list`, will always return 1 frame.
```
emacsclient --eval '(frame-list)'
(#<frame F1 0xa3c680>)
```

When an real frame is open, two frames are reported by the `frame-list`
function.
```
emacsclient --eval '(frame-list)'
(#<frame F1 0xa3c680> #<frame emacs Prelude - ~/oh-my-zsh/plugins/emacs/emacsclient.sh 0xf50e10>)
```

See:
https://emacs.stackexchange.com/questions/44537/extra-frame-in-visible-frame-list-when-started-in-daemon-mode-is-causing-prob
https://emacs.stackexchange.com/questions/18859/dont-let-the-daemon-frame-make-a-file-visible

Diff

 1diff --git a/plugins/emacs/emacsclient.sh b/plugins/emacs/emacsclient.sh
 2index b4f49e12b197c53061706df3c88be5a61fee917c..26b28d495b6a63a8b88c1b0c056864a200a7e8d3 100755
 3--- a/plugins/emacs/emacsclient.sh
 4+++ b/plugins/emacs/emacsclient.sh
 5@@ -5,7 +5,7 @@ _emacsfun()
 6     # get list of emacs frames.
 7     frameslist=`emacsclient --alternate-editor '' --eval '(frame-list)' 2>/dev/null | egrep -o '(frame)+'`
 8 
 9-    if [ "$(echo "$frameslist" | sed -n '$=')" -ge 1 ] ;then
10+    if [ "$(echo "$frameslist" | sed -n '$=')" -ge 2 ] ;then
11         # prevent creating another X frame if there is at least one present.
12         emacsclient --alternate-editor "" "$@"
13     else