Re: APR_INT32_MAX used by mod_delate-2.2.29

2014-11-06 Thread Jeff Trawick
On Sat, Nov 1, 2014 at 6:03 PM, Yann Ylavic ylavic@gmail.com wrote:

 mod_deflate in httpd-2.2.29 is using APR_INT32_MAX which is only
 available since APR-1.3.

 However httpd-2.x seems to require APR-1.2 only
 (http://httpd.apache.org/docs/2.2/install.html#requirements).

 Should we apply something like :

 Index: modules/filters/mod_deflate.c
 ===
 --- modules/filters/mod_deflate.c(revision 1635743)
 +++ modules/filters/mod_deflate.c(working copy)
 @@ -47,6 +47,10 @@
  #define APR_WANT_STRFUNC
  #include apr_want.h

 +#ifndef APR_INT32_MAX
 +#define APR_INT32_MAX 0x7fff
 +#endif
 +
  #include zlib.h

  static const char deflateFilterName[] = DEFLATE;
 ?

 Regards,
 Yann.


+1 here

(babbling about generic concerns with using a level of APR that we haven't
used in years has been removed)

-- 
Born in Roswell... married an alien...
http://emptyhammock.com/


Re: APR_INT32_MAX used by mod_delate-2.2.29

2014-11-06 Thread Yann Ylavic
On Thu, Nov 6, 2014 at 2:26 PM, Jeff Trawick traw...@gmail.com wrote:
 On Sat, Nov 1, 2014 at 6:03 PM, Yann Ylavic ylavic@gmail.com wrote:

 mod_deflate in httpd-2.2.29 is using APR_INT32_MAX which is only
 available since APR-1.3.

 However httpd-2.x seems to require APR-1.2 only
 (http://httpd.apache.org/docs/2.2/install.html#requirements).

 Should we apply something like :

 Index: modules/filters/mod_deflate.c
 ===
 --- modules/filters/mod_deflate.c(revision 1635743)
 +++ modules/filters/mod_deflate.c(working copy)
 @@ -47,6 +47,10 @@
  #define APR_WANT_STRFUNC
  #include apr_want.h

 +#ifndef APR_INT32_MAX
 +#define APR_INT32_MAX 0x7fff
 +#endif
 +
  #include zlib.h

  static const char deflateFilterName[] = DEFLATE;
 ?

 Regards,
 Yann.


 +1 here

 (babbling about generic concerns with using a level of APR that we haven't
 used in years has been removed)

OK, patch against 2.2.x (only) proposed by r1637104.


APR_INT32_MAX used by mod_delate-2.2.29

2014-11-01 Thread Yann Ylavic
mod_deflate in httpd-2.2.29 is using APR_INT32_MAX which is only
available since APR-1.3.

However httpd-2.x seems to require APR-1.2 only
(http://httpd.apache.org/docs/2.2/install.html#requirements).

Should we apply something like :

Index: modules/filters/mod_deflate.c
===
--- modules/filters/mod_deflate.c(revision 1635743)
+++ modules/filters/mod_deflate.c(working copy)
@@ -47,6 +47,10 @@
 #define APR_WANT_STRFUNC
 #include apr_want.h

+#ifndef APR_INT32_MAX
+#define APR_INT32_MAX 0x7fff
+#endif
+
 #include zlib.h

 static const char deflateFilterName[] = DEFLATE;
?

Regards,
Yann.