bug#18728: 24.0.50; gnus hangs while reading mail from imap server

[tsugutomo.enami@jp.sony.com]

[-- Attachment #1: Type: text/plain, Size: 442 bytes --]

Ted Zlatanov <tzz@lifelogs.com> writes:

> Could you prepare a patch against Emacs trunk or (preferably) Gnus
> master[1], incorporating Andreas' suggestion and with a ChangeLog entry?
> I'll apply it to Gnus master and it will make its way to Emacs trunk
> shortly thereafter.

Ok, patch against today's gnus.git is attached.  I've tested by changing
nnimap-keepalive and nnimap-keepalive-timer to send noop command more
frequently.

enami.

[-- Attachment #2: nnimap-wait-for-response fix to ignore NOOP response. --]
[-- Type: text/plain, Size: 851 bytes --]

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 43c5bfc..2156301 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-10-24  enami tsugutomo  <tsugutomo.enami@jp.sony.com>
+
+	* nnimap.el (nnimap-wait-for-response): Ignore NOOP response requested
+	to keep connection open.
+
 2014-10-20  Glenn Morris  <rgm@gnu.org>
 
 	* Merge in all changes up to 24.4 release.
diff --git a/lisp/nnimap.el b/lisp/nnimap.el
index ad48d47..7992151 100644
--- a/lisp/nnimap.el
+++ b/lisp/nnimap.el
@@ -1888,7 +1888,7 @@ Return the server's response to the SELECT or EXAMINE command."
 			(while (and (not (bobp))
 				    (progn
 				      (forward-line -1)
-				      (looking-at "\\*"))))
+				      (looking-at "\\*\\|[0-9]+ OK NOOP"))))
 			(not (looking-at (format "%d .*\n" sequence)))))
 	    (when messagep
 	      (nnheader-message-maybe