apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@attglobal.net>
Subject Re: cvs commit: apr-util/test testpass.c .cvsignore Makefile.in
Date Thu, 29 May 2003 11:53:33 GMT
Joe Orton wrote:
> On Wed, May 28, 2003 at 04:45:46AM -0000, Justin Erenkrantz wrote:
>>jerenkrantz    2003/05/27 21:45:46
>>  Modified:    .        CHANGES configure.in
>>               build    apu-hints.m4
>>               crypto   apr_md5.c
>>               test     .cvsignore Makefile.in
> I looked into moving the macro but there was a nasty gotcha, maybe this
> is what you found too: the glibc-style crypt_r implementation relies on
> -D_GNU_SOURCE being in CPPFLAGS, but any additions to CPPFLAGS made by
> the apr-util configure script are dropped on the floor. (since
> apr-util/crypto/Makefile just includes apr_rules.mk from APR)

I don't recall anybody encountering this, but its a great heads up.

Certainly it is appropriate for aprutil configure to be able to add 
stuff to CFLAGS/CPPFLAGS, and that should be fixed, but my own feeling 
about feature set selection symbols like _GNU_SOURCE is that they should 
be defined in apr_hints.m4.  In general, changing at mid-stream which 
flavor of system interfaces we want or which set of system interfaces we 
want can lead to inconsistencies w.r.t. any config tests done prior to 
the point that we start defining the symbol.

View raw message