On Oct 24, 2006, at 8:43 AM, Peter Eisch wrote:

When the plugin was changed to use the qpsmtp transaction temp file, this
broke the tmp_dir option.  The patch below fixes this bug [....]
[ ... http://beta.nntp.perl.org/group/perl.qpsmtpd/5681 ...]

Hi Peter,

I don't understand how the patch fixed the tmp_dir option. Unless I'm missing something then you just made the option required (and removed the sanity checking (directory exists)).

FWIW I agree with John that it'd be better to make Qpsmtpd::temp_ {file,dir} take a path parameter.


 - ask

--
http://askask.com/  - http://develooper.com/


Reply via email to