talloc dep for talloc.h

Jelmer Vernooij jelmer at vernstok.nl
Tue Jan 4 03:43:31 MST 2011


On Tue, 2011-01-04 at 21:27 +1100, Andrew Bartlett wrote:
> On Tue, 2011-01-04 at 09:55 +0100, Jelmer Vernooij wrote:
> > On Tue, 2011-01-04 at 05:40 +0100, Andrew Bartlett wrote:
> > > commit 640028d158583825ea5ffd1266f099cf8776db5d
> > > Author: Andrew Bartlett <abartlet at samba.org>
> > > Date:   Tue Jan 4 14:41:38 2011 +1100
> > > 
> > >     nsswitch Add talloc depencency for nsstest
> > >     
> > >     nsstest does not use talloc, but it includes talloc.h via includes.h,
> > >     and so without this it cannot find the right internal header.
> > >     
> > >     This wasn't noticed before, as most linux developer boxes have a talloc.h.
> > >     
> > >     The issue was reported by Adam Tauno Williams <awilliam at whitemice.org>
> > I'm a bit puzzled by this, as e.g. sn doesn't have talloc.h installed
> > either and neither have most of the build farm hosts. Perhaps it happens
> > if talloc.h is installed but not in /usr/include ?
> 
> That is odd, but I reproduced it locally, by removing libtalloc-devel
> from my Fedora 12 laptop.  I've tested it before and after this commit
> (including a waf configure).
That is odd indeed. Is talloc.pc perhaps part of the the libtalloc
package on RedHat instead of libtalloc-devel ?

Cheers,

Jelmer



More information about the samba-cvs mailing list