Seema Alevoor wrote: > I have updated the webrev ( http://cr.opensolaris.org/~seema/6670736/ ). > > Main changes include patches to apr_hints.m4 and apr_common.m4 > apr_hints.m4 - to add the threaded flags. Attached is an updated patch to solve: * We can't change default for --enable-threads * Hide stderr from "$CC -flags" to avoid noise when building with gcc * (Existing issue) Don't mess with large-file settings, because: ** Adding ?-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64? isn't necessary. APR >= 1.something already does ?-D_LARGEFILE64_SOURCE? by default. ** This type of compile setting is a potential ABI change. If we did this here and APR didn't already do it, we'd be incompatibile with apps written for APR and compiled against a normal APR build. (In fact, a Sun-distributed httpd broke the ABI of its Apache 2.0/APR 0.9 with such a change.)
Any idea why we need both -mt and _POSIX_PTHREAD_SEMANTICS? > > apr_common.m4 - to use the Sun compiler's flag, "-xM" to generate > makefile dependencies. Since, for httpd we use the bundled configure > script, there is a need to patch this file, too. > Without this, mod_auth_gss build caused the following error: > cc1: error: invalid option `t' still a mystery to me how gcc was invoked :( -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: thread_flags.txt URL: <http://mail.opensolaris.org/pipermail/webstack-discuss/attachments/20080814/d32cdca4/attachment.txt>
