Subject: bug#22414: 24.5; GNUS - No such function: nnregistry-request-group

From: Björn Bidar <[email-masked]>
Date: Tue, 04 Feb 2025 18:09:36 +0200

Robert Pluim <[email-masked]> writes:

>>>>>> On Tue, 04 Feb 2025 15:11:05 +0200, Björn Bidar <[email-masked]> said:
>     >> 
>     >> So what I hope you mean is:
>     >> 
>     >> nnregistry doesnʼt actually handle servers or groups, however we need
>     >> to define a dummy function for it in order to satisfy the
>     >> gnus-check-group-server function.
>     >> 
>     >> Robert
>
>     Björn> That does sound a little condescending to say you hope what
>     Björn> I am saying.
>
> It was not intended that way, I was trying to express what I
> understood.

OK np.

>
>     Björn> I'm not a native speaker but I think I can speak well enough to be
>     Björn> understandable. 
>     Björn> Yes that is essentially what I'm trying to say but I want to highlight
>     Björn> that you take the sentence out of context. The full message was:
>
>     Björn>      Add function for gnus-check-group-server.  nnregistry doesn't handle
>     Björn>      any servers or groups define the dummy function so Gnus-registry
>     Björn>      works as refer article method.
>
>     Björn> Did you read the first sentence? Maybe it should have been:
>
>     Björn>      Add function for gnus-check-group-server function.  nnregistry doesn't handle
>     Björn>      any servers or groups define the dummy function so Gnus-registry
>     Björn>      works as refer article method.
>
> My difficulty is because this bit: "nnregistry doesn't handle any
> servers or groups" runs into this bit: "define the dummy function so
> Gnus-registry works as refer article method" without any intervening
> junction phrase like "so we must" or similar.

Ah! Ok that makes your point much easier to get. That a connecting
phrase or word was needed was not obvious to me.

Please check the patch below:

>From 05648f89680f7beb1ee7e9d72553878d268dcc28 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Bidar?= <[email-masked]>
Date: Wed, 22 Jan 2025 01:28:22 +0200
Subject: [PATCH] Add nnregistry-request-group function for gnus-refer-article

* lisp/gnus/nnregistry.el (nnregistry-request-group):
nnregistry doesn't handle any servers or groups; however we have to
define the dummy function so Gnus-registry works as refer article
method, as the `gnus-check-group-server' function requires it.
The latter function is called when referring to articles to check the
source group first before actually requesting any articles.
---
 lisp/gnus/nnregistry.el | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lisp/gnus/nnregistry.el b/lisp/gnus/nnregistry.el
index edf773209d6..10363560ba6 100644
--- a/lisp/gnus/nnregistry.el
+++ b/lisp/gnus/nnregistry.el
@@ -47,6 +47,9 @@ nnregistry-status-message
 (deffoo nnregistry-open-server (_server &optional _defs)
   gnus-registry-db)
 
+(deffoo nnregistry-request-group (_group &optional _server _dont-check _info)
+  t)
+
 (defvar nnregistry-within-nnregistry nil)
 
 (deffoo nnregistry-request-article (id &optional _group _server buffer)
-- 
2.45.2