gnus 5.6.11

gnus 5.6.11

[Rolf Marvin B|e Lindgren]

suddenly, Gnus merges all my mail, from all my mail spools, into
nnml+private:mail.misc.  

I need a Gnus filter for post-filtering.  and a day to manually sift the
mail I've gotten the last three days.

-- 

       Rolf Lindgren           |       "The opinions expressed above are
       Sofienberggt. 13b       |        not necessarily those of anyone"
       N-0551 OSLO             |               roffe@ask.uio.no 
.

Re: gnus 5.6.11

[Justin Sheehy]

Rolf Marvin B|e Lindgren <roffe@morgoth.uio.no> writes:

> suddenly, Gnus merges all my mail, from all my mail spools, into
> nnml+private:mail.misc.  

Yep.

This is rather serious, and seems to have hit lots of people.  On
first look, it seems that the only change that could have caused it is
the new regexp-quoting in nnmail.el. 

I haven't yet managed to figure out exactly why that's making groups
not match, though.  Then again, I haven't had much time to try.

-- 
Justin Sheehy

In a cloud bones of steel.
  

Re: gnus 5.6.11

[Hrvoje Niksic]

Justin Sheehy <justin@linus.mitre.org> writes:

> Rolf Marvin B|e Lindgren <roffe@morgoth.uio.no> writes:
> 
> > suddenly, Gnus merges all my mail, from all my mail spools, into
> > nnml+private:mail.misc.  
> 
> This is rather serious, and seems to have hit lots of people.  On
> first look, it seems that the only change that could have caused it
> is the new regexp-quoting in nnmail.el.

I introduced the new regexp-quoting (Gnus failed on Windows without
it), so I feel responsible.  Although the changes are generally fine,
I see one that's not.  Try this patch:


1998-06-05  Hrvoje Niksic  <hniksic@srce.hr>

	* nnmail.el (nnmail-get-split-group): Don't regexp-quote
 	nnmail-procmail-suffix.

--- nnmail.el.orig	Fri Jun  5 11:37:23 1998
+++ nnmail.el	Fri Jun  5 11:37:34 1998
@@ -722,7 +722,7 @@
 				      (file-name-as-directory
 				       nnmail-procmail-directory)))
 				"\\([^/]*\\)"
-				(regexp-quote nnmail-procmail-suffix) "$")
+				nnmail-procmail-suffix "$")
 			(expand-file-name file))
 	  (let ((procmail-group (substring (expand-file-name file)
 					   (match-beginning 1)


-- 
Hrvoje Niksic <hniksic@srce.hr> | Student at FER Zagreb, Croatia
--------------------------------+--------------------------------
By any chance, have you seen a summoned 9th order fire elemental
wandering around?  No?  Oh..  Tell me if you do.

Re: gnus 5.6.11

[Justin Sheehy]

Hrvoje Niksic <hniksic@srce.hr> writes:

> Try this patch:

Yep, seems to do the trick.

-Justin

Re: gnus 5.6.11

[Rolf Marvin B|e Lindgren]

[Hrvoje Niksic]

| I introduced the new regexp-quoting (Gnus failed on Windows without
| it), so I feel responsible.  Although the changes are generally fine,
| I see one that's not.  Try this patch:

if this is how nnmail-get-split-group is supposed to be, it still
doesn't work. 

(defun nnmail-get-split-group (file group)
  "Find out whether this FILE is to be split into GROUP only.
If GROUP is non-nil and we are using procmail, return the group name
only when the file is the correct procmail file.  When GROUP is nil,
return nil if FILE is a spool file or the procmail group for which it
is a spool.  If not using procmail, return GROUP."
  (if (or (eq nnmail-spool-file 'procmail)
          nnmail-use-procmail)
      (if (string-match (concat "^" (regexp-quote
                                     (expand-file-name
                                      (file-name-as-directory
                                       nnmail-procmail-directory)))
                                "\\([^/]*\\)"
                                (regexp-quote nnmail-procmail-suffix) "$")
                        (expand-file-name file))
          (let ((procmail-group (substring (expand-file-name file)
                                           (match-beginning 1)
                                           (match-end 1))))
            (if group
                (if (string-equal group procmail-group)
                    group
                  nil)
              procmail-group))
        nil)
    group))


-- 

       Rolf Lindgren           |       "The opinions expressed above are
       Sofienberggt. 13b       |        not necessarily those of anyone"
       N-0551 OSLO             |               roffe@ask.uio.no 
.

Re: gnus 5.6.11

[Hrvoje Niksic]

Rolf Marvin B|e Lindgren <roffe@morgoth.uio.no> writes:

> [Hrvoje Niksic]
> 
> | I introduced the new regexp-quoting (Gnus failed on Windows without
> | it), so I feel responsible.  Although the changes are generally fine,
> | I see one that's not.  Try this patch:
> 
> if this is how nnmail-get-split-group is supposed to be, it still
> doesn't work.

It's not.  You didn't apply my patch successfully.

>       (if (string-match (concat "^" (regexp-quote
>                                      (expand-file-name
>                                       (file-name-as-directory
>                                        nnmail-procmail-directory)))
>                                 "\\([^/]*\\)"
>                                 (regexp-quote nnmail-procmail-suffix) "$")
                                   ^^^^^^^^^^^^ <-- bogus regexp-quote

-- 
Hrvoje Niksic <hniksic@srce.hr> | Student at FER Zagreb, Croatia
--------------------------------+--------------------------------
"Listen, kid, we're all in it together!" -- Harry Tuttle in _Brazil_

Re: gnus 5.6.11

[Lars Magne Ingebrigtsen]

Hrvoje Niksic <hniksic@srce.hr> writes:

> I introduced the new regexp-quoting (Gnus failed on Windows without
> it), so I feel responsible.  Although the changes are generally fine,
> I see one that's not.  Try this patch:

Thanks for the patch; I've applied it to Gnus v5.6.12.

-- 
(domestic pets only, the antidote for overdose, milk.)
  larsi@gnus.org * Lars Magne Ingebrigtsen

Re: gnus 5.6.11

[Bruce Stephens]

Rolf Marvin B|e Lindgren <roffe@morgoth.uio.no> writes:

> suddenly, Gnus merges all my mail, from all my mail spools, into
> nnml+private:mail.misc.  
> 
> I need a Gnus filter for post-filtering.  and a day to manually sift the
> mail I've gotten the last three days.

Ditto.  I don't see anything in the changelog to explain this.  I'm
doing what the manual tells me to do.  (Sobs quietly.)  Please release
a fix, Lars.  Please!