Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 22 Nov 2011 21:41:55 +0100
From:      Jilles Tjoelker <jilles@stack.nl>
To:        Dag-Erling Smorgrav <des@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules
Message-ID:  <20111122204154.GA14090@stack.nl>
In-Reply-To: <201111211640.pALGedXg051270@svn.freebsd.org>
References:  <201111211640.pALGedXg051270@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Nov 21, 2011 at 04:40:39PM +0000, Dag-Erling Smorgrav wrote:
> Author: des
> Date: Mon Nov 21 16:40:39 2011
> New Revision: 227798
> URL: http://svn.freebsd.org/changeset/base/227798

> Log:
>   Simplify the libpam build by removing the shared modules' dependency
>   on the shared library.  The modules are loaded by the library, so we
>   know it'll be there when we need it.

Although this will work, I think it trades the quality of the binaries
for a cleaner build system. It is better to pass all libraries to ld(1)
even though a .so may have unresolved references: the NEEDED entry
serves as an additional protection against version mismatches and symbol
versioning (if you ever add it) requires ld(1) to have access to the .so
containing the definition so it knows the version name to store in the
output file.

-- 
Jilles Tjoelker



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20111122204154.GA14090>