Date: Wed, 06 Mar 2002 19:15:04 -0500 From: "Brian F. Feldman" <green@FreeBSD.org> To: Mark Murray <mark@grondar.za> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libpam/modules modules.inc src/lib/libpam/modules/pam_alreadyloggedin Makefile pam_alreadyloggedin.8 pam_alreadyloggedin.c Message-ID: <200203070015.g270F4A43670@green.bikeshed.org> In-Reply-To: Message from Mark Murray <mark@grondar.za> of "Wed, 06 Mar 2002 23:29:32 GMT." <200203062329.g26NTWRV065417@grimreaper.grondar.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Mark Murray <mark@grondar.za> wrote: > > > Reviewed by: nobody > > > Approved by: nobody > > > > Robert took a look at it, and other people agreed it was a generally > > interesting module to have. Add to that that it's small, not turned on > > unless you do it yourself, and doesn't break the build, and I don't see what > > the problem is adding a new PAM module. > > I suspect that the pam_self module should take the responsibility > for doing what this does, probably with the assistance of some > appropriate options. I'm not sure; I think it's probably worth keeping them logically separate since they don't perform very similar actions, although they allow a somewhat similar policy to be implemented. > I have a (minor) problem with the name; it's too clumsy. Yeah, but I would like to know of a better one (without just moving the ugliness to the name of the option used ;) Thanks for checking it out! -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org <> bfeldman@tislabs.com \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\ 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?200203070015.g270F4A43670>