[SSSD] [PATCH] LDAP: Don't use macro _XOPEN_SOURCE for extra features

Jakub Hrozek jhrozek at redhat.com
Mon May 26 14:52:28 UTC 2014


On Sat, May 24, 2014 at 07:38:17PM +0200, Lukas Slebodnik wrote:
> On (24/05/14 19:10), Lukas Slebodnik wrote:
> >ehlo,
> >
> >We defined macro _XOPEN_SOURCE before time.h, because we need function strptime
> >The problem is with undef after including header time.h
> >The macro _XOPEN_SOURCE can be defined on some platforms
> >and undef can cause problems.
> >
> >We detect all necessary feature macros in configure script
> >using AC_USE_SYSTEM_EXTENSIONS or AC_GNU_SOURCE.
> >It is better to include header file config.h
> >instead of defining macro _XOPEN_SOURCE
> >
> >LS
> 
> attached patch with small change.
> 
> LS

Similar to other patches in the mini-thread, I only tested by building
on F-20 and RHEL-6. The build went fine, no new errors or warnings.

ACK



More information about the sssd-devel mailing list