Opened 4 years ago
Closed 4 years ago
#60304 closed defect (fixed)
Fix nginx rtmp module
| Reported by: | tobiasps (Tobias Steinmann) | Owned by: | neverpanic (Clemens Lang) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.6.2 |
| Keywords: | haspatch | Cc: | Schamschula (Marius Schamschula) |
| Port: | nginx |
Description
The rtmp variant on the nginx port fails to build. Attached is a patch to fix the issue. The patch updates the version of nginx-rtmp-module to current v1.2.1. With this version the patch-ngx_rtmp_core_module.c.diff file is not needed.
Note that due to the fact that the http_push module has the exact same version and the exact same filename (v1.2.1.tar.gz), I had to change the master site to workaround this conflict. There may be a better way to solve this? Is there a way to define a different filename on the macport mirrors than on the master site?
Attachments (1)
Change History (5)
Changed 4 years ago by tobiasps (Tobias Steinmann)
| Attachment: | Portfile-nginx-rtmp.diff added |
|---|
comment:1 Changed 4 years ago by jmroot (Joshua Root)
| Cc: | Schamschula added |
|---|---|
| Owner: | set to neverpanic |
| Status: | new → assigned |
comment:2 Changed 4 years ago by mf2k (Frank Schima)
| Keywords: | haspatch added; rtmp removed |
|---|---|
| Type: | update → defect |
comment:3 Changed 4 years ago by Schamschula (Marius Schamschula)
Nginx is somewhat unique in the number of source files, and their numbering convention. Your work around is likely the best solution.
comment:4 Changed 4 years ago by Schamschula (Marius Schamschula)
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Patch for using the nginx-rtmp-module version 1.2.1. This fixes building. The version and sources filename conflicts with http_push and therefore the master site has been changed as a workaround.