On Thu, Feb 10, 2011 at 02:46:50PM -0500, Simo Sorce wrote:
On Thu, 10 Feb 2011 15:58:51 +0100
Sumit Bose <sbose(a)redhat.com> wrote:
> > > Please add
> > >
> > > --- a/Makefile.am
> > > +++ b/Makefile.am
> > > @@ -638,6 +638,7 @@ auth_tests_SOURCES = \
> > > $(SSSD_UTIL_OBJ)
> > > auth_tests_CFLAGS = \
> > > $(AM_CFLAGS) \
> > > + -DSYSDB_TEST \
> > > $(CHECK_CFLAGS)
> > > auth_tests_LDADD = \
> > > $(SSSD_LIBS) \
> > >
> > > to make auth-tests work again.
> >
> > Isn't this a separate issue ?
> > Or should it be addressed in the context of this patch ?
>
> I haven't seen this issues with older libldb so I think is it
> reasonable to handle both together.
Patch updated.
Fixes build and tests, thank you
ACK
bye,
Sumit
Simo.
--
Simo Sorce * Red Hat, Inc * New York