Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-23 Thread Dag-Erling Smørgrav
Kostik Belousov writes: > Does plugin depend on the library which loaded it ? In other words, does > it reference the symbols from the library ? If yes, then plugin _must_ > have a dependency on the library recorded as DT_NEEDED. Yes. I would really, really prefer to find a solution that does no

Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-22 Thread Kostik Belousov
On Tue, Nov 22, 2011 at 10:38:02PM +0100, Dag-Erling Sm??rgrav wrote: > Dag-Erling Sm??rgrav writes: > > Jilles Tjoelker writes: > > > 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 th

Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-22 Thread Dag-Erling Smørgrav
Dag-Erling Smørgrav writes: > Jilles Tjoelker writes: > > 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 addi

Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-22 Thread Dag-Erling Smørgrav
Jilles Tjoelker writes: > 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 misma

Re: svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-22 Thread Jilles Tjoelker
On Mon, Nov 21, 2011 at 04:40:39PM +, 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.

svn commit: r227798 - in head: . lib/libpam lib/libpam/modules

2011-11-21 Thread Dag-Erling Smorgrav
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