Thanks for the heads up on that.  I guess that's what I get for writing
code and not testing it completely.  I have made the change to the file
on the site so anybody else that gets it will get a working version.

BTW.  When I was testing altermime, I noticed that it does not add the
disclaimer to the bottom of text only emails.  Is anybody else seeing
this?  I called it like this:

altermime --disclaimer=/tmp/altermime.txt
-input=/var/vpopmail/domains/domain/account/mymessage

However, it adds the disclaimer fine to multipart mime messages.

-- 
Thanks,
Ed McLain

On Wed, 2004-02-18 at 19:05, Jyri wrote:
> Hi everyone!
> 
> Loads of thanks to Ed for providing this patch! =)
> 
> After a little tweaking it seems to be working just great. Just remember
> to turn off $skip_text_msgs, otherwise disclaimer will not be added to
> text-only messages. I suggest Ed's Altermime thingy should be included
> to the next release of Q-S, with necessary changes so that Altermime
> will be run even if $skip_text_msgs is set.
> 
> There appears to be a small bug in the altermime sub. The problem is
> this line:
> 
>   $alter_options="--input=$scandir/$wmaildir/new/$file_id";
> 
> On this line, previously set alter_options are wiped away. As a result,
> Altermime is called without --disclaimer option, resulting in absolutely
> nothing. ;)
> 
> I made it work by removing the above line and adding the --input option
> to the previous if-then-else structure:
> 
> if ( $altermime_text && $altermime_html ) {
>         $alter_options = "--disclaimer=$altermime_text \
>            --disclaimer-html=$altermime_html \
>            --input=$scandir/$wmaildir/new/$file_id";
>   } else {
>         $alter_options = "--disclaimer=$altermime_text \
>         --htmltoo --input=$scandir/$wmaildir/new/$file_id";
>   }
> 
> Seems to work just beautifully now. =)
> 
> - Jyri
> 
> 
> 
> 
> -------------------------------------------------------
> SF.Net is sponsored by: Speed Start Your Linux Apps Now.
> Build and deploy apps & Web services for Linux with
> a free DVD software kit from IBM. Click Now!
> http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
> _______________________________________________
> Qmail-scanner-general mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/qmail-scanner-general




-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
_______________________________________________
Qmail-scanner-general mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/qmail-scanner-general

Reply via email to