On Wed, Feb 08, 2006 at 02:41:28PM +0100, martin f krafft wrote:
> tags 339661 + patch
> thanks
> 
> Here's the patch. Note, I am not a perl dude. I did test the
> functionality, but you never know. However, it integrates well, the
> un/subscribe commands can be used together with others (separate
> mails will be sent), but I don't vouch for it.

Thanks for this patch.

I don't understand why you've introduced this $btsserver variable in
all of the initial variables.  If we want the bts script to be usuable
on other servers, we could do that, but then we'd need to introduce a
command-line option/config file variable to handle this.  In the
meantime, why not just have

$btssubscribeemail = '[EMAIL PROTECTED]';
$btsunsubscribeemail = '[EMAIL PROTECTED]';

Also, many users won't be able to run /usr/sbin/sendmail -f, so that's
not helpful.  If the subscription system can't handle faked From:
lines, then we're stuffed.

   Julian


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to