You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rj...@apache.org on 2017/10/15 14:05:39 UTC

svn commit: r1812219 - /httpd/httpd/branches/2.4.x/STATUS

Author: rjung
Date: Sun Oct 15 14:05:39 2017
New Revision: 1812219

URL: http://svn.apache.org/viewvc?rev=1812219&view=rev
Log:
Revert.

Modified:
    httpd/httpd/branches/2.4.x/STATUS

Modified: httpd/httpd/branches/2.4.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1812219&r1=1812218&r2=1812219&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Sun Oct 15 14:05:39 2017
@@ -124,31 +124,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
      2.4.x patch: trunk works
      +1: jorton, wrowe, jim
 
-  *) configure.in: Latest macOS Xcode 9 installs clang-900, which more
-     fully honors strict-prototypes, which causes AC_CHECK_LIB to
-     fail when --enable-maintainer-mode is set. Also affects clang 5.
-     trunk patch: http://svn.apache.org/r1810448
-                  http://svn.apache.org/r1810998
-     2.4.x patch: svn merge -c 1810448,1810998 ^/httpd/httpd/trunk .
-     +1: jim, wrowe, rjung
-     [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: The following two comments are not meant to block applying that change.
-            - A possible enhancement would be to detect the problem, e.g.
-            using AC_CHECK_LIB for a test that should always succeed and
-            if it fails, try again with "-Wno-error=strict-prototypes".
-            - Another more far reaching enhancement would be to add
-            "-Wno-error=strict-prototypes" only during configure but not
-            later when we do the actual build in order to still detect missing
-            prototypes then. Since configure does not seem to have the
-            concept of settings only applied during configure run, we would
-            need to remember "-Wno-error=strict-prototypes" as a flag we just
-            added to make configure work. Then, right before configure
-            generates output files, we would strip any such flag from CFLAGS
-            to generate the correct build time CFLAGS.
-            Unfortunately things might get trickier when building apr/apu
-            in combination with httpd.
-
 
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
@@ -233,6 +208,19 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
                   (merge without changes to motorz)
      +1: ylavic, jim
 
+  *) configure.in: Latest macOS Xcode 9 installs clang-900, which more
+     fully honors strict-prototypes, which causes AC_CHECK_LIB to
+     fail when --enable-maintainer-mode is set. Also affects clang 5.
+     trunk patch: http://svn.apache.org/r1810448
+                  http://svn.apache.org/r1810998
+     2.4.x patch: svn merge -c 1810448,1810998 ^/httpd/httpd/trunk .
+     +1: jim, wrowe
+     -1: rjung
+     [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 separate mail to the dev list.
+
 
 PATCHES/ISSUES THAT ARE BEING WORKED
   [ New entries should be added at the START of the list ]