Re: svn commit: r1768245 - /httpd/httpd/trunk/modules/cache/mod_socache_memcache.c

2016-11-06 Thread William A Rowe Jr
On Sun, Nov 6, 2016 at 1:45 PM, Jim Jagielski wrote: > Why? We trust the apr_memcache.c returns the correct > response... No need to check the version of APR or > APU since mod_socache_memcache.c works fine no > matter what version is used. I missed that this is a trunk/2.4 httpd enhancement, n

Re: mod_socache_redis

2016-11-06 Thread Hank Ibell
Hello again Jim, Thanks for the quick response. All is well now with the latest updates. Thanks! Regards, Hank Ibell On Sun, Nov 6, 2016 at 3:21 PM, Jim Jagielski wrote: > Somehow things got out of sync. Should be fixed > now. > > > On Nov 6, 2016, at 2:59 PM, Jacob Champion wrote: > > > > On

Re: mod_socache_redis

2016-11-06 Thread Jim Jagielski
Somehow things got out of sync. Should be fixed now. > On Nov 6, 2016, at 2:59 PM, Jacob Champion wrote: > > On 11/06/2016 05:32 AM, Jim Jagielski wrote: >> This is weird... Are you using the stuff at the ASF?? >> The format warning are way wrong as well: >> >>/** Keyspace hits */ >>apr

Re: mod_socache_redis

2016-11-06 Thread Jacob Champion
On 11/06/2016 05:32 AM, Jim Jagielski wrote: This is weird... Are you using the stuff at the ASF?? The format warning are way wrong as well: /** Keyspace hits */ apr_uint64_t keyspace_hits; /** Keyspace misses */ apr_uint64_t keyspace_misses; keyspace_* are obviously not apr_uin

Re: svn commit: r1768245 - /httpd/httpd/trunk/modules/cache/mod_socache_memcache.c

2016-11-06 Thread Jim Jagielski
Why? We trust the apr_memcache.c returns the correct response... No need to check the version of APR or APU since mod_socache_memcache.c works fine no matter what version is used. > On Nov 6, 2016, at 11:58 AM, William A Rowe Jr wrote: > > > On Sun, Nov 6, 2016 at 10:57 AM, William A Rowe Jr

Re: svn commit: r1768245 - /httpd/httpd/trunk/modules/cache/mod_socache_memcache.c

2016-11-06 Thread William A Rowe Jr
On Sun, Nov 6, 2016 at 10:57 AM, William A Rowe Jr wrote: > Wrap this up in an #if APU_MAJOR_VERSION > 1 || APU_MINOR_VERSION > 6 > > before backporting? (Yes - these symbols will still exist in unifed APR2 > and later.) > #if APU_MAJOR_VERSION > 1 || APU_MINOR_VERSION >= 6 (My bad, sorry.) It

Re: svn commit: r1768245 - /httpd/httpd/trunk/modules/cache/mod_socache_memcache.c

2016-11-06 Thread William A Rowe Jr
Wrap this up in an #if APU_MAJOR_VERSION > 1 || APU_MINOR_VERSION > 6 before backporting? (Yes - these symbols will still exist in unifed APR2 and later.) Since we can't demand users update apr to pick up 2.4.next, and some will be using the OS-deployed APR. On Nov 5, 2016 11:47 AM, wrote: >

Re: mod_socache_redis

2016-11-06 Thread Jim Jagielski
This is weird... Are you using the stuff at the ASF?? The format warning are way wrong as well: /** Keyspace hits */ apr_uint64_t keyspace_hits; /** Keyspace misses */ apr_uint64_t keyspace_misses; keyspace_* are obviously not apr_uint32_ts > On Nov 6, 2016, at 1:11 AM, Hank Ibel

Re: mod_socache_redis

2016-11-06 Thread Jim Jagielski
Is this Xcode8 or 7? I'm on OSX 10.11.6 and Xcode 7.3.1 > On Nov 6, 2016, at 1:11 AM, Hank Ibell wrote: > > Hello Jim, > > I am trying to build trunk, but I am getting a compiler error. It seems the > structure apr_redis_stats_t is missing a rejected_connections field. I was > able to compil

Bug report for Apache httpd-2 [2016/11/06]

2016-11-06 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned