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 Ibell <[email protected]> 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 compile by adding the missing field to apr_redis_stats_t of course.
> 
> Here is the compiler output (built on macOS):
> 
> mod_socache_redis.c:332:29: error: no member named 'rejected_connections' in 
> 'apr_redis_stats_t'
>                      stats->rejected_connections);
>                      ~~~~~  ^
> mod_socache_redis.c:334:22: warning: format specifies type 'unsigned long' 
> but the argument has type 'apr_uint32_t' (aka 'unsigned int') [-Wformat]
>                      stats->keyspace_hits, stats->keyspace_misses);
>                      ^~~~~~~~~~~~~~~~~~~~
> mod_socache_redis.c:334:44: warning: format specifies type 'unsigned long' 
> but the argument has type 'apr_uint32_t' (aka 'unsigned int') [-Wformat]
>                      stats->keyspace_hits, stats->keyspace_misses);
>                                            ^~~~~~~~~~~~~~~~~~~~~~
> mod_socache_redis.c:361:28: error: no member named 'rejected_connections' in 
> 'apr_redis_stats_t'
>                     stats->rejected_connections, br);
>                     ~~~~~  ^
> mod_socache_redis.c:363:21: warning: format specifies type 'unsigned long' 
> but the argument has type 'apr_uint32_t' (aka 'unsigned int') [-Wformat]
>                     stats->keyspace_hits, stats->keyspace_misses, br);
>                     ^~~~~~~~~~~~~~~~~~~~
> mod_socache_redis.c:363:43: warning: format specifies type 'unsigned long' 
> but the argument has type 'apr_uint32_t' (aka 'unsigned int') [-Wformat]
>                     stats->keyspace_hits, stats->keyspace_misses, br);
>                                           ^~~~~~~~~~~~~~~~~~~~~~
> 
> On Sat, Nov 5, 2016 at 6:24 PM, Jim Jagielski <[email protected]> wrote:
> Trunk now includes mod_socache_redis, which requires APR 1.6
> and above. Please test it out and provide feedback :)
> 
> I also added in the required mod_status hook info for mod_socache_memcache
> as well, so now it was also provide mod_status info. I plan to
> also submit that for backporting to 2.4 as well ;)
> 

Reply via email to