Loading STATUS +1 −13 Original line number Diff line number Diff line Loading @@ -213,19 +213,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK: [This seems to fit into the mold of per-platform quirks which we process in CTR flow, adding my +1 to note that the patch looks sane.] rjung: I think we need this also for GCC, not only recent clang. See my below suggestion (r1812263) *) configure.in: Fix maintainer mode with GCC/Clang. Setting -Wstrict-prototypes in combination with -Werror leads to compiler errors during configure checks (autoconf generates incomplete prototypes). Adding -Wno-error=strict-prototypes lets the compiler tolerate those. Possible future enhancement: remember such "configure time only" flags and remove them from CFLAGS before generating our build time files (Makefile, config_vars.mk etc.), so that the full -Werror is in place during building. trunk patch: http://svn.apache.org/r1812263 2.4.x patch: svn merge -c r1812263 ^/httpd/httpd/trunk . +1: rjung See the dev list discusion about using NOTEST_CFLAGS. PATCHES/ISSUES THAT ARE BEING WORKED Loading Loading
STATUS +1 −13 Original line number Diff line number Diff line Loading @@ -213,19 +213,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK: [This seems to fit into the mold of per-platform quirks which we process in CTR flow, adding my +1 to note that the patch looks sane.] rjung: I think we need this also for GCC, not only recent clang. See my below suggestion (r1812263) *) configure.in: Fix maintainer mode with GCC/Clang. Setting -Wstrict-prototypes in combination with -Werror leads to compiler errors during configure checks (autoconf generates incomplete prototypes). Adding -Wno-error=strict-prototypes lets the compiler tolerate those. Possible future enhancement: remember such "configure time only" flags and remove them from CFLAGS before generating our build time files (Makefile, config_vars.mk etc.), so that the full -Werror is in place during building. trunk patch: http://svn.apache.org/r1812263 2.4.x patch: svn merge -c r1812263 ^/httpd/httpd/trunk . +1: rjung See the dev list discusion about using NOTEST_CFLAGS. PATCHES/ISSUES THAT ARE BEING WORKED Loading