[dev] Re: ingo procmail patch

[Ben Chavet]

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

Quoting Martin Luethi <horde at humbapa.ch>:

> Mon, 08 Mar 2004 11:49:46 +0000 Ronny Adsetts
> <ronny.adsetts at amazinginternet.com>:
>> I already submitted a patch some time ago that was applied that allowed
>> spaces in folder names with procmail filters by simply quoting the folder
>> name string.
>>
>> I suspect this patch needs to be reverted - not had time to do a CVS update
>> and check it out yet though...
>
> your patch works fine for me :-)
>
> but version 1.42 of ingo/lib/Script/procmail.php has:
> return '"$DEFAULT/.' . str_replace(' ', '\ ', $folder) . '/"';
>
> replacing " " with "\ " does not work on my system (procmail v3.22 on Linux)
> procmail creates a new folder with "\" in its name
>
> I think the best would be to add a new "scriptparams-option" in
> config/backends.php. e.g.:
> 'escapespaces' => false
>
>
> g. martin luethi
> --
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org


-------------- next part --------------
A non-text attachment was scrubbed...
Name: procmail.php.diff
Type: text/x-patch
Size: 662 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20040308/ec672ee2/procmail.php.bin