Re: [Patch] tell expiry-hook functions where expired messages are going

[Alan Schmitt <alan.schmitt@polytechnique.org>]

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

Hello,

I gave this patch a try (as it's been applied to the git repo and I've
been able to use it) and I'm seeing an error when messages are refiled
during expiration.

Here is the backtrace:

Debugger entered--Lisp error: (wrong-type-argument symbolp "nnimap+zimbra:archive")
  fboundp("nnimap+zimbra:archive")
  gnus-summary-expire-articles()
  run-hooks(gnus-summary-prepare-exit-hook)
  apply(run-hooks gnus-summary-prepare-exit-hook)
  gnus-run-hooks(gnus-summary-prepare-exit-hook)
  gnus-summary-exit()
  call-interactively(gnus-summary-exit nil nil)

The problematic string corresponds to the target mailbox for the
expiration:

#+begin_src emacs-lisp
  (setq gnus-parameters
        '(("^old.*"
           (total-expire . t)
           (expiry-wait . 1)
           (expiry-target . "nnimap+zimbra:archive"))
          ...
          ("^work$"
           (total-expire . t)
           (expiry-wait . 7)
           (expiry-target . "nnimap+zimbra:archive"))
          ...
          ))

#+end_src

Looking at the code, the error comes from the patched code. Shouldn't
there be a check that `nnmail-expiry-target' is a symbol before calling
fboundp?

Thanks,

Alan

-- 
OpenPGP Key ID : 040D0A3B4ED2E5C7

[-- Attachment #2: Type: application/pgp-signature, Size: 494 bytes --]