[PATCH] nnimap-split-articles

[PATCH] nnimap-split-articles

[Danny Siu]

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


Fixed nnimap-split-articles so that it won't cause error when calling
nnmail-cache-insert.

Can someone with CVS write access commit the patch for me?

Thanks!
-- 
Danny Dick-Fung Siu        mailto:dsiu@adobe.com
Acrobat Engineering @ Adobe Systems Incorporated

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: imap.patch --]
[-- Type: text/x-patch, Size: 679 bytes --]

Index: lisp/ChangeLog
===================================================================
RCS file: /usr/local/cvsroot/gnus/lisp/ChangeLog,v
retrieving revision 6.1365
diff -r6.1365 ChangeLog
0a1,5
> 2002-04-29  Danny Siu  <dsiu@adobe.com>
> 
> 	* nnimap.el (nnimap-split-articles): Calls nnmail-cache-insert with 
> 	single argument.
> 
Index: lisp/nnimap.el
===================================================================
RCS file: /usr/local/cvsroot/gnus/lisp/nnimap.el,v
retrieving revision 6.29
diff -r6.29 nnimap.el
2a3,6
> ; $Revision: $ 
> ; $Date: $ 
> ; $Author: $ 
> ; $Nokeywords: $ 
1191c1195
< 						   "message-id") to-group)))
---
> 						   "message-id"))))

Re: [PATCH] nnimap-split-articles

[Simon Josefsson]

Danny Siu <dsiu@adobe.com> writes:

> Fixed nnimap-split-articles so that it won't cause error when calling
> nnmail-cache-insert.
>
> Can someone with CVS write access commit the patch for me?
...
>> 	* nnimap.el (nnimap-split-articles): Calls nnmail-cache-insert with 
>> 	single argument.
...
> 1191c1195
> < 						   "message-id") to-group)))
> ---
>> 						   "message-id"))))

The function takes two argument here:

nnmail.el: (defun nnmail-cache-insert (id &optional group)

Are you perhaps using nnmail.el from non-CVS?

Re: [PATCH] nnimap-split-articles

[David S Goldberg]

That's not what I see.  I even rm'd lisp/nnmail.el and redid cvs up
-d.  nnmail.el still shows only ID as a valid argument.
-- 
Dave Goldberg
david.goldberg6@verizon.net

Re: [PATCH] nnimap-split-articles

[Simon Josefsson]

David S Goldberg <david.goldberg6@verizon.net> writes:

> That's not what I see.  I even rm'd lisp/nnmail.el and redid cvs up
> -d.  nnmail.el still shows only ID as a valid argument.

Oops!  Sorry, I didn't commit the nnmail.el fix.  Should be done now.
Sorry about that.

Re: [PATCH] nnimap-split-articles

[Danny Siu]

yeah!  it works now.

Simon Josefsson writes:

  Simon> David S Goldberg <david.goldberg6@verizon.net> writes:
  >> That's not what I see.  I even rm'd lisp/nnmail.el and redid cvs up -d.
  >> nnmail.el still shows only ID as a valid argument.

  Simon> Oops!  Sorry, I didn't commit the nnmail.el fix.  Should be done
  Simon> now.  Sorry about that.

-- 
Danny Siu