On Wed, Aug 30, 2006 at 02:09:32PM +0200, Stefan Hornburg wrote:
> >>| Same as it's always been.  The Courier build gets the mailbox location 
> >>| from environment variables set by Courier.  The standalone build uses 
> >>| the -d option, and courier-authlib.
> >
> >I don't see any configure options about this. Could it be that the same
> >binary supports both modes?
> 
> This code in maildrop/configure.in distinguishes standalone and Courier
> version:
> 
> if test -d $srcdir/../courier
> then
> #
> # This version of maildrop is integrated into Courier mail server
> 
>       AC_DEFINE_UNQUOTED(HAVE_COURIER,1,
>       [ Whether this version of maildrop is part of Courier ])
> fi

Ah, right. I grepped further for HAVE_COURIER, and it seems to me that
all remaining instances of that could fairly easily be replaced by run-time
options. The bulk of it is in maildrop/main.C, and it surrounds a couple of
chunks of Courier-related code.

We already eliminated this one due to it being a semantic error:

--- a/maildrop/deliver.C
+++ b/maildrop/deliver.C
@@ -153,11 +153,7 @@ Buffer     b;
                if ( deliver_maildir.MaildirOpen(mailbox, deliver_file,
                        maildrop.msgptr->MessageSize()) < 0)
                {
-#if HAVE_COURIER
                        throw 75;
-#else
-                       throw 77;
-#endif
                }
 
                format_mbox.Init(0);

There's an analogous one like that in maildir/deliverquota.c where it throws
an error that mail quota is exceeded, which sounds more like something that
should be modifiable by sysadmin.

-- 
     2. That which causes joy or happiness.


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

Reply via email to