search

masterzen / nginx-upload-progress-module

Nginx module implementing an upload progress system, that monitors RFC1867 POST uploads as they are transmitted to upstream servers.
http://wiki.codemongers.com/NginxHttpUploadProgressModule
Other
432 stars 101 forks source link

FTBFS with nginx 1.1.15 on Debian #20

Closed davromaniak closed 12 years ago

davromaniak commented 12 years ago

Hi.

I tried to upgrade the nginx packages using the just released nginx 1.1.15, and the build failed.

Here is the log extract : /tmp/buildd/nginx-1.1.15/debian/modules/nginx-upload-progress/ngx_http_uploadprogress_module.c: In function 'ngx_http_track_uploads': /tmp/buildd/nginx-1.1.15/debian/modules/nginx-upload-progress/ngx_http_uploadprogress_module.c:1445:26: error: 'NGX_PARSE_LARGE_TIME' undeclared (first use in this function) /tmp/buildd/nginx-1.1.15/debian/modules/nginx-upload-progress/ngx_http_uploadprogress_module.c:1445:26: note: each undeclared identifier is reported only once for each function it appears in make[2]: *\ [objs/addon/nginx-upload-progress/ngx_http_uploadprogress_module.o] Error 1

Apparently, they removed the NGX_PARSE_LARGE_TIME variable starting 1.1.15.

Could you take a look at fixing it ?

Thank you very much.

perusio commented 12 years ago

Indeed it has been removed. Here's a quick fix.

Gist here and pull request.

masterzen commented 12 years ago

This is fixed in 03cbf1faf3e7aec147f807bb771b8f3405bd761e. Thank you for your report.

davromaniak commented 12 years ago

It works well.

Thanks for your reactivity.