[dev][patch] Re: ingo procmail patch

[Ben Chavet]

I don't really think this is necessary.  The reason my patch broke things is
because I didn't notice the quotes on the maildir part & because I don't use
maildir, I didn't catch it until somebody mentioned it.  We should use either
quotes or escape the spaces, but not both.

The patch you submitted a while back fixed maildir, but not mbox.  My patch
takes care of mbox, but I broke maildir in the process (sorry 'bout that).  My
most recent submission should fix things, but it hasn't been committed yet.

--Ben

Quoting Martin Luethi <horde at humbapa.ch>:

> attached is a small patch who adds a new "scriptparams-option" in
> ingo/config/backends.php:
>
> // Whether the spaces (' ') in folder names should be escaped
> // ('\ ') or not. Default is to not escape spaces.
> 'escape_spaces' => false,
>
> g. martin luethi
>
>
> Mon,  8 Mar 2004 09:30:54 -0600 Ben Chavet <ben at chavet.net>:
>> I'm running procmail 3.22 with postfix using mbox.  before my patch, I had a
>> rule to go to mail/Virus Alerts.  Whenever this rule matched, 
>> though, it would
>> create a mail/Virus folder and put the message in there.  Using mail/Virus\
>> Alerts took care of this.
>>
>> I didn't test the patch on a maildir setup because I don't have one readily
>> available, I just assumed that it would behave the same...apparently I was
>> wrong :).  The attached patch reverts the maildir portion of my
>> original patch.
>>
>> --Ben