yaoweibin / nginx_ajp_module

support AJP protocol proxy with Nginx
http://github.com/yaoweibin/nginx_ajp_module
246 stars 60 forks source link

Build failure with nginx 1.7.9 #33

Closed jbergstroem closed 9 years ago

jbergstroem commented 9 years ago

There's been some refactoring of the upstream module data types (well, more than that), which makes the build fail:

nginx_ajp_module-0.3.0/ngx_http_ajp_module.c:870:53: error: invalid type argument of '->' (have 'int')
     if (conf->upstream.cache && conf->upstream.cache->data == NULL) {
yaoweibin commented 9 years ago

Thank you. I will check this problem.

在 2014年12月27日,下午1:13,Johan Bergström notifications@github.com 写道:

There's been some refactoring of the upstream module data types (well, more than that), which makes the build fail:

nginx_ajp_module-0.3.0/ngx_http_ajp_module.c:870:53: error: invalid type argument of '->' (have 'int') if (conf->upstream.cache && conf->upstream.cache->data == NULL) { — Reply to this email directly or view it on GitHub https://github.com/yaoweibin/nginx_ajp_module/issues/33.

jbergstroem commented 9 years ago

FWIW, I started looking at what @PiotrSikora was doing to fix cache_purge: https://github.com/FRiCKLE/ngx_cache_purge/commit/065f0cae56fc5720db636885ce5b2fbaf6f09f8c

itpp16 commented 9 years ago

Have a look at my attempt, https://github.com/yaoweibin/nginx_ajp_module/pull/34

jbergstroem commented 9 years ago

fwiw, this applies to 1.7.10 as well.

yaoweibin commented 9 years ago

I have fixed the compatible problem. Try the latest revision in the master branch.

Thanks.