Joseph Mingrone <jrm@ftfl.ca> writes:

> On Thu, 2021-08-05 at 19:54, Alexandr Vityazev <avityazev@posteo.org> wrote:
>
>> On 2021-08-05, 10:15 -0700, Eric Abrahamsen <eric@ericabrahamsen.net> wrote:
>
>>> Eric Abrahamsen <eric@ericabrahamsen.net> writes:
>
>
>>> [...]
>
>>>> I'm getting to it! I just need to find a moment to sit down and set up
>>>> some dummy data and test the various possible configurations. It
>>>> shouldn't be to hard to unilaterally force prefixed group names where
>>>> that's needed, and force unprefixed names elsewhere, I just need to
>>>> find the time. I'd like to get this fixed once and for all.
>
>>> Okay, I think I've finally got it. The incoming group names are indeed
>>> prefixed or not depending on whether the server is primary or not
>>> (arguably this should be fixed to always unprefixed in nnselect, but
>>> I'll worry about that later), so this patch enforces unprefixed group
>>> names within the function body, and prefixed group names in the return
>>> value. Please give it a shot!
>
>>> Eric
>
>
>
>> The patch was applied to the master branch and it doesn't work for me.
>> In my case because of line 1388 with the code:
>
>> #+begin_src elisp 
>> (setq group (gnus-group-full-name group server))
>> #+end_src
>
>
>> code on 1395 line:
>
>> #+begin_src elisp
>> (nnmaildir-base-name-to-article-number
>> 		   (substring article 0 (string-match ":" article))
>> 		   group (string-remove-prefix "nnmaildir:" server))
>> #+end_src
>
>
>> always return nil, because the group on line 1397 always gets something
>> like "nnmaildir+gmail:Inbox" and not "Inbox".
>
>> And in the code on 1400 line:
>
>> #+begin_src elisp
>> (member group groups)
>> #+end_src
>
>
>> also requires the group similar to "Inbox", so it can only be solved by
>> deleting line 1388:
>
>> #+begin_src elisp
>> (setq group (gnus-group-full-name group server))
>> #+end_src
>
> I see the same as you Alexandr, i.e., with the patch applied to a recent
> commit from the master branch, still no search results, but after
> removing line 1388, results are returned.

Grr, sorry, I sent the wrong diff -- the original call to
`gnus-group-full-name' was still in there. I promise I'm sending the
correct, freshly-tested diff this time!