Date: Fri, 07 Aug 2009 13:54:32 -0700 From: "Kevin Oberman" <oberman@es.net> To: "b. f." <bf1783@googlemail.com> Cc: freebsd-current@FreeBSD.org Subject: Re: Unable to build HEAD Message-ID: <20090807205432.8FA4D1CC31@ptavv.es.net> In-Reply-To: Your message of "Thu, 06 Aug 2009 11:37:50 -0000." <d873d5be0908060437ifb69d7dn59f37373b53f818e@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> Date: Thu, 6 Aug 2009 11:37:50 +0000 > From: "b. f." <bf1783@googlemail.com> > > On 8/6/09, Kevin Oberman <oberman@es.net> wrote: > >I have tested a patch from bf and it works. I've asked if he wants to > >submit the PR or if he wants me to. If I don;t hear from him, I'll > >submit tomorrow. > > Slightly revised and augmented patch is in: > > http://www.freebsd.org/cgi/query-pr.cgi?pr=137483 I think the patch is right, but I am still broken. I also had to remove the ".if ${MK_OPENSSH) != "no" and paired ".endif" from /usr/src/lib/libpam/modules/modules.inc. Once this was done, it looks like everything is correct. I think the right answer is to either unconditionally build the pam module or to add an option that is specific to the module. I think the former is really the way to go as the module only adds 46K to the system and, if you build without OpenSSH, you are either building an embedded system where you will almost certainly be trimming a lot further than the src.conf file allows, or because you are using the version from ports. If the latter, you almost certainly WILL want pam_ssh. And thanks for fixing the Makefile and submitting the PR! -- R. Kevin Oberman, Network Engineer Energy Sciences Network (ESnet) Ernest O. Lawrence Berkeley National Laboratory (Berkeley Lab) E-mail: oberman@es.net Phone: +1 510 486-8634 Key fingerprint:059B 2DDF 031C 9BA3 14A4 EADA 927D EBB3 987B 3751
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090807205432.8FA4D1CC31>