Opened 14 years ago

Closed 14 years ago

#25871 closed update (fixed)

rtmpdump 2.3

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: CaptSolo
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: kthenriksson@…
Port: rtmpdump

Description

rtmpdump should be updated to 2.3. Attached is the beginning of a patch to do that, but it doesn't compile. There is a thread in their mailing list discussing problems compiling and suggesting patches, but the patches do not appear to relate to version 2.3 of the software. It seems apparent to me that the developers of rtmpdump have no conception of how to release software that can be used, which is unfortunate.

Attachments (3)

2.3.diff (1.3 KB) - added by ryandesign (Ryan Carsten Schmidt) 14 years ago.
beginnings of a patch
rtmpdump2.3.patch (1.3 KB) - added by kthenriksson@… 14 years ago.
rtmpdump-2.2_to_2.3.svn.diff (4.5 KB) - added by michaelld (Michael Dickens) 14 years ago.
"svn diff" in rtmpdump mports directory, upgrade from 2.2 to 2.3 with correct dylib numbering and creation

Download all attachments as: .zip

Change History (8)

Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: 2.3.diff added

beginnings of a patch

comment:1 Changed 14 years ago by jmroot (Joshua Root)

Type: defectupdate
Version: 1.9.1

comment:2 Changed 14 years ago by kthenriksson@…

Cc: kthenriksson@… added

Cc Me!

Changed 14 years ago by kthenriksson@…

Attachment: rtmpdump2.3.patch added

comment:3 Changed 14 years ago by kthenriksson@…

This minor change to your patch allows rtmpdump 2.3 to properly build and install.

Changed 14 years ago by michaelld (Michael Dickens)

"svn diff" in rtmpdump mports directory, upgrade from 2.2 to 2.3 with correct dylib numbering and creation

comment:4 Changed 14 years ago by michaelld (Michael Dickens)

I just attached a more extensive set of patches if you're interested. These get the dylib version numbering and creation correct, which is not provided in the original Makefile. These also correct the 'man' install location, and have the executables link against the dylib instead of the .a library. With these patches, rtmpdump 2.3 compiles and installs nicely, and works for me.

comment:5 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Thanks. I committed a modified version in r77989.

Note: See TracTickets for help on using tickets.