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.inI 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.