Showstopper to patch adoption, along with a more general objection

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1747027 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index 9da200974ddee68e6e1eba289d53836402d6c68a..7dcde7984da304bb9879b41a7a0c5e07af8f3ce3 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -203,7 +203,20 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
      2.4.x patch: http://home.apache.org/~jailletc36/apr_cstr_casecmp_v2.diff
      jailletc36: v2: update as per discussion on dev@. Do not mix ap_ and apr_
                  namespaces + tweak DOXYGEN comments.
-     +1: jailletc36, icing 
+     +1: jailletc36, icing
+     -0: wrowe [Prefer that we pre-@deprecate this API and encourage users
+                to adopt the apr_ convention, reversing the #defines, being
+                ready for the httpd 2.next minor release.]
+     -1: wrowe [Notes this patch creates binary incompatible versions of httpd
+                with this change; there must be #else stubs in util.c sources
+                compiled against APR 1.6+ for binary compatibility, even if
+                these are simply;
+                AP_DECLARE(int) ap_cstr_casecmpn(const char *s1,
+                                                 const char *s2, apr_size_t n)
+                {
+                    return apr_cstr_casecmpn(s1, s2, n);
+                }
+               ]
 
   *) abs: Add OPENSSL_Applink in Visual Studio >= 2015 or abs fails with a no
      OPENSSL_Applink error. PR59630