Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#38968 closed defect (fixed)

nginx 1.4.0 update/build fails

Reported by: jonasjonas (Frank Hellenkamp) Owned by: neverpanic (Clemens Lang)
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc: cooljeanius (Eric Gallager)
Port: nginx

Description

The update for nginx 1.4.0 fails (with the following options): nginx-1.4.0_0+mp4+realip+ssl+upload+upload_progress

The problem seems to be in upload module: no member named 'to_write' in 'ngx_http_request_body_t'

I attached the build log.

Attachments (1)

main.log (51.2 KB) - added by jonasjonas (Frank Hellenkamp) 11 years ago.
build log

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by jonasjonas (Frank Hellenkamp)

Attachment: main.log added

build log

comment:1 Changed 11 years ago by neverpanic (Clemens Lang)

Owner: changed from macports-tickets@… to cal@…
Status: newassigned

It seems the upload_progress module isn't compatible with nginx 1.4.0. There's no newer upstream version of this module, though, so there's not much I can do.

comment:2 Changed 11 years ago by jonasjonas (Frank Hellenkamp)

After some research it seems that the upload-module also wont be updated to nginx 1.4.0 for the time being.

I will keep an eye on this. If the module won't be updated for a longer time, I would recommend removing the +upload variant from the portfile.

There seems to be a different module from a different maintainer, which would provide a similar functionality: https://github.com/pgaertig/nginx-big-upload - But i haven't tested it yet.

The +upload_progress module seems to work fine though.

Thanks for your feedback.

comment:3 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:4 Changed 11 years ago by larryv (Lawrence Velázquez)

#39773 seems to have a fix for this for 1.4.1.

comment:5 Changed 11 years ago by neverpanic (Clemens Lang)

Resolution: fixed
Status: assignedclosed

Fixed using the patch from #39773. Revbump was avoided, since I will update nginx to 1.4.2 shortly.

comment:6 Changed 11 years ago by neverpanic (Clemens Lang)

Fix was in r108280.

Note: See TracTickets for help on using tickets.