Re: Gnus and notmuch

[Eric Abrahamsen <eric@ericabrahamsen.net>]

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

Alexis Praga <alexis.praga@gmail.com> writes:

> Eric Abrahamsen <eric@ericabrahamsen.net> writes:
>> Well, one problem at a time :)
>>
>> While it should be possible to use a single notmuch installation to
>> search two nnmaildir servers, as you're doing, it's also possible that
>> this is the source of the problem. Could you also show me your
>> ~/.notmuch-config?
>>
>> These two lines are the source of the problem: gnus-search was able to
>> find the article numbers from search hits in the "archive" group, but
>> was not able to turn search hits in the "home.alex.mail.free.archive"
>> group into article numbers. Usually this means it wasn't actually able
>> to locate the files on disk, which usually means there's a mismatch with
>> the 'remove-prefix setting.
>>
>> From the traceback, it looks like you were only searching the
>> nnmaildir+gmail:archive group. Do you have any idea why you were getting
>> results from home.alex.mail.free.archive?
> I don'tt really know why it is searching for the other maildir. That's
> why I have been restarting emacs and removing ~/.newrrc.el*, hoping it
> would resolve some conflict with my previous attempts.

I think see the problem, it's yet more fragility in parsing the output
of search results.

Notmuch is returning all results in all directories, and Gnus filters
out the results it wants depending on which paths match the group name.
Because "archive" is a substring of "home.alex.mail.free.archive", it
thinks it's a relevant search result.

I guess I'm not entirely sure why it fails to find the articles from
that group -- I would expect the articles from the other group to be
located correctly -- but this is probably still the source of the
problem.

This stuff is so fragile that I dread to think of what else might break
with this change, but... If you're comfortable patching and
re-evaluating, would you please apply the attached, then re-evaluate the
`gnus-search-indexed-parse-output' method, and see if that fixes the
problem?

Another proper solution (in addition to this) would be to figure out how
to pass the groups parameter to notmuch in either its folder: or path:
key. That would avoid duplicate work, and presumably run faster. I'll
put this on the todo list.

Eric



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: gnus-search-groups.diff --]
[-- Type: text/x-patch, Size: 971 bytes --]

diff --git a/lisp/gnus/gnus-search.el b/lisp/gnus/gnus-search.el
index fc9f8684f6..71a65ab95d 100644
--- a/lisp/gnus/gnus-search.el
+++ b/lisp/gnus/gnus-search.el
@@ -1347,15 +1347,18 @@ gnus-search-indexed-parse-output
 						server query &optional groups)
   (let ((prefix (slot-value engine 'remove-prefix))
 	(group-regexp (when groups
-			(mapconcat
-			 (lambda (group-name)
-			   (mapconcat #'regexp-quote
-				      (split-string
-				       (gnus-group-real-name group-name)
-				       "[.\\/]")
-				      "[.\\\\/]"))
-			 groups
-			 "\\|")))
+                        (concat
+                         "\\`"
+			 (mapconcat
+			  (lambda (group-name)
+			    (mapconcat #'regexp-quote
+				       (split-string
+				        (gnus-group-real-name group-name)
+				        "[.\\/]")
+				       "[.\\\\/]"))
+			  groups
+			  "\\|")
+                         "\\'")))
 	artlist vectors article group)
     (goto-char (point-min))
     (while (not (or (eobp)