[jakob at crt.se: Re: [PATCH] Maildir support]

[pete at more.net]

I don't think he's talking about mail server or clients here.

I think this needs some clairification.  This really only effects
the $MAIL upon login.  I've messed with this some, because it's caused
problems for me personally at work.  

Here's the deal with an example:

If you do a configure, on a Solaris 2.7 box, and then build sshd, when you log
in you get this is you echo $MAIL, MAIL=/var/mail//<username> .  These two
slashes get botched because of the way Sun defines MAILDIR in maillock.h

#define MAILDIR         "/var/mail/"

So, this (even though broke in Sun's case because of the trailing "/"), is
what the guy is talking about.  This example is for the Mbox not Maildir mail format which how mail
is stored.  So in this case we have the same problem though using different
formats.    The MAIL variable NEEDS the right info.  This is not feature
creep.  It's just another (abeit., awesome!) way that systems can store mail,
and many new mail servers and mail clients support it.  If that MAIL variable
is set wrong.  This needs to be fixed and patched for both formats.  

As he said in the original patch, whether you want to support the MAIL var is
another question, but if you're going to, it should be done correctly for both
mbox and mailder.

By the way, can someone fix the configure for Solaris, so that it doesn't use this?
#define MAILDIR         "/var/mail/"






----- Forwarded message from Jakob Schlyter <jakob at crt.se> -----

Delivered-To: openssh-unix-dev-list-93873 at shitei.mindrot.org
Date: Mon, 13 Aug 2001 21:05:59 +0200 (MEST)
From: Jakob Schlyter <jakob at crt.se>
To: Martin Johansson <martin at fatbob.nu>
Cc: <openssh-unix-dev at mindrot.org>
Subject: Re: [PATCH] Maildir support
In-Reply-To: <20010813210249.A15239 at fatbob.nu>
Precedence: bulk

On Mon, 13 Aug 2001, Martin Johansson wrote:

> On Mon, Aug 13, 2001 at 05:00:22PM +0200, Jakob Schlyter wrote:
>
> > why should OpenSSH even bother checking mail? isn't that what your
> > mailreader is for? I think this is featurism.
>
> Well, it already does (see the configuration keyword CheckMail in
> sshd's man page), but only for mbox style mailboxes. The patch does
> not add a feature but tweaks an already existing feature to work
> with Maildir style mailboxes.

should we also add support for MH, MBX and - why not - imap or pop?

	jakob

----- End forwarded message -----

-- 
Pete Greenwell 
System Administrator  
Missouri Research and Education Network [MOREnet]  
http://www.more.net