diff options
| author |   Michael Orlitzky <mjo@gentoo.org> | 2017-06-18 09:43:45 -0400 |
|---|---|---|
| committer | Michael Orlitzky <mjo@gentoo.org> | 2017-08-11 11:25:25 -0400 |
| commit | 8f36aaf184fd8aeb5387d51f22246179b6dd349c (patch) | |
| tree | 257ce608956f2f78ed9955045fdf040e465db0ac /eclass | |
| parent | media-tv/mythtv: Drop 0.27.6 (diff) | |
| download | gentoo-8f36aaf184fd8aeb5387d51f22246179b6dd349c.tar.gz gentoo-8f36aaf184fd8aeb5387d51f22246179b6dd349c.tar.bz2 gentoo-8f36aaf184fd8aeb5387d51f22246179b6dd349c.zip | |
apache-2.eclass: don't enable maintainer-mode with USE=debug.
The --enable-maintainer-mode flag serves two purposes: first, it
enables compile-time warnings; and second, it forces all compiled
modules to be loaded at runtime. To achieve the first goal, the build
system checks for the "-std=c89" compiler flag and appends it to
CFLAGS if found. This can cause a problem later, because the CFLAGS
used to build apache are used by apxs to build extension modules.
In particular, if you have an extension that relies on C99 or some
later standard, and if apache was built with --enable-maintainer-mode,
then apxs will attempt to build that extension according to C89 and
fail. This failure can be observed in in bugs 621686, 621756, and
621794 when apache is built with USE=debug, because USE=debug sets
the --enable-maintainer-mode flag.
Since there is not much reason for an end user to enable maintainer-
mode, the simplest fix for this is the wholesale removal of the
--enable-maintainer-mode flag from USE=debug. This prevents us from
having to append CFLAGS to each extension package that uses a later
standard, or from having to add a USE dependency forbidding USE=debug
in apache (which would create conflicts with e.g. mod_backtrace).
Gentoo-Bug: 621686
Gentoo-Bug: 621756
Gentoo-Bug: 621794
Diffstat (limited to 'eclass')
| -rw-r--r-- | eclass/apache-2.eclass | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/eclass/apache-2.eclass b/eclass/apache-2.eclass index 78a7c5ce474d..dbd55abe8c90 100644 --- a/eclass/apache-2.eclass +++ b/eclass/apache-2.eclass @@ -1,4 +1,4 @@ -# Copyright 1999-2015 Gentoo Foundation +# Copyright 1999-2017 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # @ECLASS: apache-2.eclass @@ -402,7 +402,7 @@ apache-2_pkg_setup() { setup_modules if use debug; then - MY_CONF+=( --enable-maintainer-mode --enable-exception-hook ) + MY_CONF+=( --enable-exception-hook ) fi elog "Please note that you need SysV IPC support in your kernel." |