Opened 13 years ago

Closed 12 years ago

#32482 closed update (fixed)

rtmpdump: update to 2.4

Reported by: dargo@… Owned by: CaptSolo
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: larryv (Lawrence Velázquez), feranick@…, ryandesign (Ryan Carsten Schmidt)
Port: rtmpdump

Description

2.4 adds support for RTMPE type 9 handshakes. this is needed for sites like hulu.

Attachments (3)

Portfile-rtmpdump-2.4.diff (1.4 KB) - added by ryandesign (Ryan Carsten Schmidt) 13 years ago.
proposed patch
patch-Makefile.diff (345 bytes) - added by ryandesign (Ryan Carsten Schmidt) 13 years ago.
patch-librtmp-Makefile.diff (766 bytes) - added by ryandesign (Ryan Carsten Schmidt) 13 years ago.

Download all attachments as: .zip

Change History (17)

comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to captsolo@…
Port: rtmpdump added
Summary: update rtmpdump to 2.4rtmpdump: update to 2.4
Version: 2.0.3

comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

I don't see any 2.4 distfile in their download directory.

comment:3 Changed 13 years ago by larryv (Lawrence Velázquez)

As far as I can tell, v2.4 is only available in the git repo.

comment:4 Changed 13 years ago by dargo@…

there's now a 2.4 source available. from their mailing list: http://lists.mplayerhq.hu/pipermail/rtmpdump/2011-October/001664.html

looks like a lot of sites are using the type 10 handshake now, so we'll want a rtmpdump 2.5 as soon as it's released :-)

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

Cc: larry.velazquez@… feranick@… added

Has duplicate #32805.

comment:6 in reply to:  4 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to dargo@…:

there's now a 2.4 source available. from their mailing list: http://lists.mplayerhq.hu/pipermail/rtmpdump/2011-October/001664.html

Bizarre. The distfile is there. But it doesn't show up in the listing of that directory.

Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: Portfile-rtmpdump-2.4.diff added

proposed patch

Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: patch-Makefile.diff added

Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: patch-librtmp-Makefile.diff added

comment:7 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Keywords: haspatch added

Attached is a Portfile patch and updated patchfiles to update to 2.4. Please evaluate and indicate whether it can be committed, or attach alternate patches.

Upstream now defines the library version number as 0; in our old patchfile, since upstream had not defined a library version number, we had set it the same as the program version number. Since with this update the library version number will change from 2.3 down (!) to 0, all ports linking with librtmpdump will need their revisions increased to force a rebuild.

Upstream changes in 2.4 include support for building OS X dylibs, and building the shared library by default (in addition to the static one) so some parts of the patchfiles were no longer necessary. Upstream neglected to use the -install_name argument when creating the dylib so I retained that from the old patch. I did not retain the -current_version and -compatibility_version arguments we were using in our old patch, because 0 is the default anyway, and I recall badness on Tiger when these arguments are set to zero. Upstream would be wise to select a library version number greater than zero.

Other dylib creation arguments in our old patchfile I did not retain are -Wl,-undefined, -Wl,dynamic_lookup, and -Wl,-single_module, because I do not know their purpose. Meanwhile, dylib creation arguments upstream is using that we were not using are -flat_namespace, -undefined suppress and -fno-common; I do not know their purpose either.

I retained the part of our patchfiles that changed the VERSION variable from e.g. "v2.4" to just "2.4". I do not know why we are doing this. I imagine it was at least partly to ease computation of the library version number; since we no longer compute the library version number, perhaps we can stop munging the VERSION.

Since this version's distfile does not show up in the directory listing, the livecheck still thinks 2.3 is the latest version. Upstream would be wise to fix their directory listing.

comment:8 Changed 12 years ago by falkoni@…

Somebody please work on this, since yle-dl requires rtmpdump ≥ 2.4 to work.

Update: Ok. I went and applied the patch, installed rtmpdump 2.4 and it seems to work, at least with yle-dl. Tried this with Snow Leopard (10.6.8).

Last edited 12 years ago by falkoni@… (previous) (diff)

comment:9 Changed 12 years ago by sami.laine@…

I agree with previous commentor. The portfile produced from diff in this ticket + additional two patch files produce a working binary. These files should be pushed to the repository in order to replace completely outdated 2.3 version (yle-dl is but one of the many scripts which require newer version).

Waiting for upstream to fix issues presented above is as smart as waiting for moon to turn into gold and drop into your pocket: as we all know, mplayer folks are notorious about how they handle things.

Last edited 12 years ago by sami.laine@… (previous) (diff)

comment:10 Changed 12 years ago by larryv (Lawrence Velázquez)

Cc: larryv@… added; larry.velazquez@… feranick@… ryandesign@… removed

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

Cc: feranick@… ryandesign@… added

Oops.

comment:12 in reply to:  7 Changed 12 years ago by larryv (Lawrence Velázquez)

Replying to ryandesign@…:

Upstream now defines the library version number as 0; in our old patchfile, since upstream had not defined a library version number, we had set it the same as the program version number. Since with this update the library version number will change from 2.3 down (!) to 0, all ports linking with librtmpdump will need their revisions increased to force a rebuild.

Which ports would require a revbump? port echo depends:rtmpdump doesn’t turn up anything for me.

comment:13 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

port echo depends:rtmpdump will only find ports that depend on rtmpdump in their default variants. To find ports that that might depend on it in a non-default variant, you can use port file all | sort -u | xargs grep :rtmpdump but that doesn't turn up any either. I guess we can update the port without needing to change any other ports.

comment:14 Changed 12 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: newclosed

Updated in r104380.

Note: See TracTickets for help on using tickets.