Re: [patch] Fix linking ldconfig when --enable-static-nss (with ChangeLog)

[Paul Pluzhnikov <ppluzhnikov at google dot com>]

On Thu, Mar 1, 2012 at 1:05 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Thursday 01 March 2012 15:42:00 Joseph S. Myers wrote:
>> On Sat, 25 Feb 2012, Paul Pluzhnikov wrote:
>> > The patch is much simpler than the one attached to
>> > http://sourceware.org/bugzilla/show_bug.cgi?id=6528, but requires that
>> > a statically-linked binary explicitly list "-lnss_files -lnss_dns
>> > -lresolv" on the link line.
>> >
>> > I *think* Daniel's patch from PR 6528 would not require clients to do
>> > that, so perhaps that's better?
>>
>> As I see it, Daniel's patch has the twin advantages:
>>
>> * It reduces the amount of code involved in supporting
>> --enable-static-nss.
>>
>> * It makes --enable-static-nss easier to use.
>>
>> Thus, I'd prefer Daniel's patch. ?What do other people think?
>
> having more .o's in libc.a isn't really a problem and would make things
> simpler, sounds good to me

Thanks.

Attached is Daniel's patch rebased to current git trunk.

I've build glibc on Linux/x86_64 with and without --enable-static-nss.

Could someone please apply it?

Thanks,
-- 
Paul Pluzhnikov

2012-03-01  Daniel Jacobowitz  <dmj@google.com>
            Paul Pluzhnikov  <ppluzhnikov@google.com>

        [BZ #6528]
        * grp/Makefile, inet/Makefile, login/Makefile, nscd/Makefile,
        posix/Makefile, pwd/Makefile, rt/Makefile, sunrpc/Makefile: Do not
        set otherlibs for static NSS.
        * nss/Makefile: Likewise.  Include libnss_files in libc.a for
        --enable-static-nss.  Remove special case for getent.
        * resolv/Makefile: Include libnss_dns in libc.a for
        --enable-static-nss.

Attachment: glibc-pr6528-20120301.txt
Description: Text document