Date: Sat, 20 Apr 2002 18:54:37 +0400 From: "Andrey A. Chernov" <ache@nagual.pp.ru> To: Robert Watson <rwatson@FreeBSD.org> Cc: "Jacques A. Vidrine" <nectar@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/crypto/openssh servconf.c session.c Message-ID: <20020420145437.GB5350@nagual.pp.ru> In-Reply-To: <Pine.NEB.3.96L.1020420104515.64976P-100000@fledge.watson.org> References: <20020420143525.GB27108@madman.nectar.cc> <Pine.NEB.3.96L.1020420104515.64976P-100000@fledge.watson.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Apr 20, 2002 at 10:46:22 -0400, Robert Watson wrote: > > On Sat, 20 Apr 2002, Jacques A. Vidrine wrote: > > > On Sat, Apr 20, 2002 at 02:26:44AM -0700, Andrey A. Chernov wrote: > > > 1) Surprisingly, "CheckMail" handling code completely removed from this > > > version, so documented "CheckMail" option exists but does nothing. > > > Bring it back to life adding code back. > > > > Why not just finish killing it? > > Yeah, personally I'd prefer we avoid introducing gratuitous diffs against > the OpenSSH source -- some changes are necessary (PAM, et al), but > anything beyond the base necessary changes just makes updating more of a > pain. We've already had updates stalled by >6-8 months because of many > local changes, so that's a problem we need to be very careful to avoid. Please note that "CheckMail" was in OpenSSH since long time and was removed just recently, making lack of "login" look & feel without any reason. Moreover, old "ssh" (pre-OpenSSH) check mail too without any option, like "login" does. Users really want to know they have new mail when they logged in! About setusercontext() changes: they are absolutely necessary, if we support login.conf. We can either remove them with whole login.conf support (why we have login.conf so?) or do it correctly. We can't live with half-singing half-dancing old variant, I mean broken login.conf support. -- Andrey A. Chernov http://ache.pp.ru/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020420145437.GB5350>