Re: [PATCH] add sendbox feature

2008-08-17 Thread Aron Griffis
Louis-David Mitterrand wrote: [Sun Aug 17 2008, 04:40:38PM EDT] > On Thu, Aug 14, 2008 at 01:51:58PM -0400, Aron Griffis wrote: > > > > Moreover it all falls apart when bouncing (remailing) > > messages, because that code path calls mutt_invoke_sendmail > > directly, and of course no

Re: [PATCH] add sendbox feature

2008-08-17 Thread Louis-David Mitterrand
On Thu, Aug 14, 2008 at 01:51:58PM -0400, Aron Griffis wrote: > > Moreover it all falls apart when bouncing (remailing) > messages, because that code path calls mutt_invoke_sendmail > directly, and of course no fcc is done. In any case bouncing can't work with courier's Outbox feature

Re: [PATCH] add sendbox feature

2008-08-15 Thread Aron Griffis
FYI, Rocco's comments on IRC: < pdmef> agriffis: For that patch I'd rather not add two new options but only $sendbox and make mutt use whatever is non-empty from the chain: $sendbox -> $smtp_url -> $sendmail < pdmef> agriffis: Also, why not simply add a function

Re: [PATCH] add sendbox feature

2008-08-15 Thread Aron Griffis
Louis-David Mitterrand wrote: [Fri Aug 15 2008, 10:36:34AM EDT] > One question: will the patch still work when using imap in tunneling mode? Yes, that's how I use it for IMAP servers that don't have Outbox enabled by default, for example: set tunnel="ssh -q imapserver 'set -a; source /etc/courie

Re: [PATCH] add sendbox feature

2008-08-15 Thread Louis-David Mitterrand
On Thu, Aug 14, 2008 at 01:51:58PM -0400, Aron Griffis wrote: > Louis-David Mitterrand wrote: [Thu Aug 14 2008, 01:05:06PM EDT] > > Why not simply use > > > > set record=imaps://your.courier.host/INBOX.Outbox > > set sendmail='cat > /dev/null' ## or equivalent > > > > No need for a patch, or di

Re: [PATCH] add sendbox feature

2008-08-14 Thread Aron Griffis
Louis-David Mitterrand wrote: [Thu Aug 14 2008, 01:05:06PM EDT] > Why not simply use > > set record=imaps://your.courier.host/INBOX.Outbox > set sendmail='cat > /dev/null' ## or equivalent > > No need for a patch, or did I miss something? I agree that seems like the obvious solution, but it ha

Re: [PATCH] add sendbox feature

2008-08-14 Thread Louis-David Mitterrand
On Wed, Aug 13, 2008 at 12:01:20PM -0400, Aron Griffis wrote: > > So this patch adds two settings: sendbox and use_sendbox. > Sendbox specifies a mailbox to which sent messages should be > saved, and use_sendbox instructs mutt to use sendbox instead of > sendmail. For example: > > set sendbo

[PATCH] add sendbox feature

2008-08-13 Thread Aron Griffis
This is a repost of my sendbox patch against mercurial tip. This patch has been previously posted twice. The first time was over two years ago. 2006-08-09 http://marc.info/?l=mutt-dev&m=115513617113648&w=2 2007-12-04 http://marc.info/?l=mutt-dev&m=119681273920794&w=2 This relatively simple patc