Gnus development mailing list
 help / color / mirror / Atom feed
* Next registry error
@ 2011-04-21  6:32 Tassilo Horn
  2011-04-21 14:58 ` Ted Zlatanov
  0 siblings, 1 reply; 6+ messages in thread
From: Tassilo Horn @ 2011-04-21  6:32 UTC (permalink / raw)
  To: ding

Hi Ted,

it seems your commit

  30ad87c1e3b03e7c0bf9a6c3b1269e753ba53955
  Add ignored groups back.

breaks my setup again.  When trying to enter some of my IMAP groups, I
get this backtrace:

--8<---------------cut here---------------start------------->8---
Debugger entered--Lisp error: (wrong-type-argument stringp ("nntp" t))
  string-match(("nntp" t) "nnimap+Fastmail:INBOX")
  byte-code("\b\304.\211.\203.\0\n@.\305	\v\"\203.\0\306\307	\"\210\nA\211.\204\b.*\304\207" [list r --cl-dolist-temp-- word nil string-match throw found] 4)
  gnus-grep-in-list("nnimap+Fastmail:INBOX" (("nntp" t) ("training" t) ("Junk" t)))
  gnus-registry-ignore-group-p("nnimap+Fastmail:INBOX")
  gnus-registry-handle-action("<INX.11c7968586b903edc1a23b429ee.22b67.1d2146.c644d.12f735a71ab194.bounce@family-newsletter.de>" nil "nnimap+Fastmail:INBOX" "Gewinn einen Tapetenwechsel im Wert von 10.000 Euro" nil nil)
  gnus-registry-register-message-ids()
  run-hooks(gnus-summary-prepare-hook)
  apply(run-hooks gnus-summary-prepare-hook)
  gnus-run-hooks(gnus-summary-prepare-hook)
  gnus-summary-prepare()
  gnus-summary-read-group-1("nnimap+Fastmail:INBOX" nil t nil nil nil)
  gnus-summary-read-group("nnimap+Fastmail:INBOX" nil t nil nil nil nil)
  gnus-group-read-group(nil t)
  gnus-group-select-group(nil)
  gnus-topic-select-group(nil)
  call-interactively(gnus-topic-select-group nil nil)
  recursive-edit()
  debug(error (wrong-type-argument stringp ("nntp" t)))
  string-match(("nntp" t) "nnimap+Fastmail:INBOX")
  byte-code("\b\304.\211.\203.\0\n@.\305	\v\"\203.\0\306\307	\"\210\nA\211.\204\b.*\304\207" [list r --cl-dolist-temp-- word nil string-match throw found] 4)
  gnus-grep-in-list("nnimap+Fastmail:INBOX" (("nntp" t) ("training" t) ("Junk" t)))
  gnus-registry-ignore-group-p("nnimap+Fastmail:INBOX")
  gnus-registry-handle-action("<INX.11c7968586b903edc1a23b429ee.22b67.1d2146.c644d.12f735a71ab194.bounce@family-newsletter.de>" nil "nnimap+Fastmail:INBOX" "Gewinn einen Tapetenwechsel im Wert von 10.000 Euro" nil nil)
  gnus-registry-register-message-ids()
  run-hooks(gnus-summary-prepare-hook)
  apply(run-hooks gnus-summary-prepare-hook)
  gnus-run-hooks(gnus-summary-prepare-hook)
  gnus-summary-prepare()
  gnus-summary-read-group-1("nnimap+Fastmail:INBOX" nil t nil nil nil)
  gnus-summary-read-group("nnimap+Fastmail:INBOX" nil t nil nil nil nil)
  gnus-group-read-group(nil t)
  gnus-group-select-group(nil)
  gnus-topic-select-group(nil)
  call-interactively(gnus-topic-select-group nil nil)
--8<---------------cut here---------------end--------------->8---

Bye,
Tassilo
-- 
Sent from my Emacs



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Next registry error
  2011-04-21  6:32 Next registry error Tassilo Horn
@ 2011-04-21 14:58 ` Ted Zlatanov
  2011-04-21 15:51   ` Tassilo Horn
  0 siblings, 1 reply; 6+ messages in thread
From: Ted Zlatanov @ 2011-04-21 14:58 UTC (permalink / raw)
  To: ding

On Thu, 21 Apr 2011 08:32:33 +0200 Tassilo Horn <tassilo@member.fsf.org> wrote: 

TH> Hi Ted,
TH> it seems your commit

TH>   30ad87c1e3b03e7c0bf9a6c3b1269e753ba53955
TH>   Add ignored groups back.

TH> breaks my setup again.  When trying to enter some of my IMAP groups, I
TH> get this backtrace:
TH> Debugger entered--Lisp error: (wrong-type-argument stringp ("nntp" t))
TH>   string-match(("nntp" t) "nnimap+Fastmail:INBOX")
TH>   byte-code("\b\304.\211.\203.\n@.\305	\v\"\203.\306\307	\"\210\nA\211.\204\b.*\304\207" [list r --cl-dolist-temp-- word nil string-match throw found] 4)
TH>   gnus-grep-in-list("nnimap+Fastmail:INBOX" (("nntp" t) ("training" t) ("Junk" t)))
TH>   gnus-registry-ignore-group-p("nnimap+Fastmail:INBOX")

Your `gnus-registry-ignored-groups' or
`nnmail-split-fancy-with-parent-ignore-groups' is 

'(("nntp" t) ("training" t) ("Junk" t)) 

but it should be a list of regexps or nil.  I don't think either of
those variables, especially `gnus-registry-ignored-groups', ever had
that format.  Can you check?

Thanks
Ted




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Next registry error
  2011-04-21 14:58 ` Ted Zlatanov
@ 2011-04-21 15:51   ` Tassilo Horn
  2011-04-21 16:37     ` Ted Zlatanov
  0 siblings, 1 reply; 6+ messages in thread
From: Tassilo Horn @ 2011-04-21 15:51 UTC (permalink / raw)
  To: Ted Zlatanov; +Cc: ding

Ted Zlatanov <tzz@lifelogs.com> writes:

Hi Ted,

> TH> Debugger entered--Lisp error: (wrong-type-argument stringp ("nntp" t))
> TH>   string-match(("nntp" t) "nnimap+Fastmail:INBOX")
> TH>   byte-code("\b\304.\211.\203.\n@.\305	\v\"\203.\306\307	\"\210\nA\211.\204\b.*\304\207" [list r --cl-dolist-temp-- word nil string-match throw found] 4)
> TH>   gnus-grep-in-list("nnimap+Fastmail:INBOX" (("nntp" t) ("training" t) ("Junk" t)))
> TH>   gnus-registry-ignore-group-p("nnimap+Fastmail:INBOX")
>
> Your `gnus-registry-ignored-groups' or
> `nnmail-split-fancy-with-parent-ignore-groups' is
>
> '(("nntp" t) ("training" t) ("Junk" t)) 

Yes,

> but it should be a list of regexps or nil.  I don't think either of
> those variables, especially `gnus-registry-ignored-groups', ever had
> that format.  Can you check?

I'm pretty sure it had that format, where t meant ignore matching groups
and nil meant don't ignore.

Oh, and the docs also use that format:

,----[ (info "(gnus)Setup") ]
|      (setq
|       gnus-registry-split-strategy 'majority
|       gnus-registry-ignored-groups '(("nntp" t)
|                                      ("nnrss" t)
|                                      ("spam" t)
|                                      ("train" t))
|       gnus-registry-max-entries 500000
|       ;; this is the default
|       gnus-registry-track-extra '(sender subject))
`----

I'm totally fine with having that variable as a list of regexps, as I
see no reason for saying ("foo" nil).  So it's just a question of you
want to make an incompatible change...

Bye,
Tassilo
-- 
Sent from my Emacs



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Next registry error
  2011-04-21 15:51   ` Tassilo Horn
@ 2011-04-21 16:37     ` Ted Zlatanov
  2011-04-22  6:53       ` Tassilo Horn
  0 siblings, 1 reply; 6+ messages in thread
From: Ted Zlatanov @ 2011-04-21 16:37 UTC (permalink / raw)
  To: ding

On Thu, 21 Apr 2011 17:51:41 +0200 Tassilo Horn <tassilo@member.fsf.org> wrote: 

TH> Debugger entered--Lisp error: (wrong-type-argument stringp ("nntp" t))
TH> string-match(("nntp" t) "nnimap+Fastmail:INBOX")
TH> byte-code("\b\304.\211.\203.\n@.\305	\v\"\203.\306\307	\"\210\nA\211.\204\b.*\304\207" [list r --cl-dolist-temp-- word nil string-match throw found] 4)
TH> gnus-grep-in-list("nnimap+Fastmail:INBOX" (("nntp" t) ("training" t) ("Junk" t)))
TH> gnus-registry-ignore-group-p("nnimap+Fastmail:INBOX")
>> 
>> Your `gnus-registry-ignored-groups' or
>> `nnmail-split-fancy-with-parent-ignore-groups' is
>> 
>> '(("nntp" t) ("training" t) ("Junk" t)) 

TH> Yes,

>> but it should be a list of regexps or nil.  I don't think either of
>> those variables, especially `gnus-registry-ignored-groups', ever had
>> that format.  Can you check?

TH> I'm pretty sure it had that format, where t meant ignore matching groups
TH> and nil meant don't ignore.

TH> Oh, and the docs also use that format:

TH> ,----[ (info "(gnus)Setup") ]
TH> |      (setq
TH> |       gnus-registry-split-strategy 'majority
TH> |       gnus-registry-ignored-groups '(("nntp" t)
TH> |                                      ("nnrss" t)
TH> |                                      ("spam" t)
TH> |                                      ("train" t))
TH> |       gnus-registry-max-entries 500000
TH> |       ;; this is the default
TH> |       gnus-registry-track-extra '(sender subject))
TH> `----

TH> I'm totally fine with having that variable as a list of regexps, as I
TH> see no reason for saying ("foo" nil).  So it's just a question of you
TH> want to make an incompatible change...

Ah.  It's a group/topic parameter, that's why it has that form, and why
I didn't see it when I did git diff against older gnus-registry.el
versions.  Sorry about that.  I made the changes so the default is given
in gnus.el and it should work OK now.  You can still set it to a list of
regexes.

Thanks
Ted




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Next registry error
  2011-04-21 16:37     ` Ted Zlatanov
@ 2011-04-22  6:53       ` Tassilo Horn
  2011-04-22 13:21         ` Ted Zlatanov
  0 siblings, 1 reply; 6+ messages in thread
From: Tassilo Horn @ 2011-04-22  6:53 UTC (permalink / raw)
  To: Ted Zlatanov; +Cc: ding

Ted Zlatanov <tzz@lifelogs.com> writes:

Hi Ted,

> TH> I'm totally fine with having that variable as a list of regexps,
> TH> as I see no reason for saying ("foo" nil).  So it's just a
> TH> question of you want to make an incompatible change...
>
> Ah.  It's a group/topic parameter, that's why it has that form, and
> why I didn't see it when I did git diff against older gnus-registry.el
> versions.  Sorry about that.  I made the changes so the default is
> given in gnus.el and it should work OK now.

It does, thank you.

> You can still set it to a list of regexes.

Yes, I can set it to a list of regexes, but that errors. ;-)

So instead of (("^nntp" t) ("training" t) ("Junk" t)) I tried using
("^nntp" "training" "Junk"), but when trying to enter a group, I got
this:

Debugger entered--Lisp error: (wrong-type-argument listp "^nntp")
  gnus-parameter-registry-ignore("nnimap+Fastmail:INBOX.Junk Mail")
  gnus-registry-register-message-ids()
  run-hooks(gnus-summary-prepare-hook)
  apply(run-hooks gnus-summary-prepare-hook)
  gnus-run-hooks(gnus-summary-prepare-hook)
  gnus-summary-prepare()
  gnus-summary-read-group-1("nnimap+Fastmail:INBOX.Junk Mail" nil t nil nil nil)
  gnus-summary-read-group("nnimap+Fastmail:INBOX.Junk Mail" nil t nil nil nil nil)
  gnus-group-read-group(nil t)
  gnus-group-select-group(nil)
  gnus-topic-select-group(nil)
  call-interactively(gnus-topic-select-group nil nil)

Bye,
Tassilo
-- 
Sent from my Emacs



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Next registry error
  2011-04-22  6:53       ` Tassilo Horn
@ 2011-04-22 13:21         ` Ted Zlatanov
  0 siblings, 0 replies; 6+ messages in thread
From: Ted Zlatanov @ 2011-04-22 13:21 UTC (permalink / raw)
  To: ding

On Fri, 22 Apr 2011 08:53:02 +0200 Tassilo Horn <tassilo@member.fsf.org> wrote: 

TH> Yes, I can set it to a list of regexes, but that errors. ;-)

TH> So instead of (("^nntp" t) ("training" t) ("Junk" t)) I tried using
TH> ("^nntp" "training" "Junk"), but when trying to enter a group, I got
TH> this:

TH> Debugger entered--Lisp error: (wrong-type-argument listp "^nntp")
TH>   gnus-parameter-registry-ignore("nnimap+Fastmail:INBOX.Junk Mail")
TH>   gnus-registry-register-message-ids()
TH>   run-hooks(gnus-summary-prepare-hook)
TH>   apply(run-hooks gnus-summary-prepare-hook)
TH>   gnus-run-hooks(gnus-summary-prepare-hook)
TH>   gnus-summary-prepare()
TH>   gnus-summary-read-group-1("nnimap+Fastmail:INBOX.Junk Mail" nil t nil nil nil)
TH>   gnus-summary-read-group("nnimap+Fastmail:INBOX.Junk Mail" nil t nil nil nil nil)
TH>   gnus-group-read-group(nil t)
TH>   gnus-group-select-group(nil)
TH>   gnus-topic-select-group(nil)
TH>   call-interactively(gnus-topic-select-group nil nil)

Should be fixed now.

Ted




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-04-22 13:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-21  6:32 Next registry error Tassilo Horn
2011-04-21 14:58 ` Ted Zlatanov
2011-04-21 15:51   ` Tassilo Horn
2011-04-21 16:37     ` Ted Zlatanov
2011-04-22  6:53       ` Tassilo Horn
2011-04-22 13:21         ` Ted Zlatanov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).