Xpra: Ticket #1227: clarify encoders and muxers in sound codec short names

When codecs were first added, there was no support for multiple muxers, but now there is... And it makes sense to use the full specification including the muxer in the codec name, so we have to switch to the new full-name and preserve backwards compatibility.

ie: vorbis will now be called vorbis+gdp and flac will now be called flac+ogg, but older clients and servers should still be able to use the old names when connecting to newer versions of xpra.

Somewhat related to #1204.


Similar to what we did for picture encodings when we switched from the old names (x264, vpx) to the new ones (h264, vp8, vp9), translate and only use the new names internally: we must send both in hello packets in case the other end doesn't understand the new names and when parsing the data (either from the command line or from a hello packet) we convert to the new names.



Sun, 12 Jun 2016 07:52:04 GMT - Antoine Martin: owner changed

Done in r12801 + r12802.

The new codecs names can be seen using xpra/sound/gstreamer_util.py, on session info and with the xpra start --speaker-codec=help / xpra start --microphone-codec=help command lines.. (and also with "attach")

If I've messed this up, it is possible that older clients or servers will be missing some codecs when they negotiate what codec to use, or worse they could fail when processing sound requests or sound packets. Tested OK here.

@afarr: this is mostly a FYI, though it wouldn't hurt to make sure that we keep backwards compatibility with older versions (0.17.x and earlier), at least for the most commonly used codecs which have been renamed by this change: vorbis -> vorbis+gdp, flac -> flac+ogg, opus -> opus+ogg, speex -> speex+ogg.


Fri, 01 Jul 2016 19:41:16 GMT - J. Max Mena: status changed; resolution set

Noted, meant to close this a while back.

I did test a 17.4 client (from /dists) against an 18.X trunk server built from source and all seemed okay. Sound was working just fine.

Closing.


Finally got that song out of my head.......


Tue, 12 Jul 2016 16:52:22 GMT - Antoine Martin: milestone changed

Milestone renamed


Thu, 10 Nov 2016 08:33:37 GMT - Antoine Martin:

A few codecs were (re-)added and the compatibility code tweaked in r14401.

Hopefully this doesn't cause any regressions.. The new codecs were added at the end of the list anyway.


Sat, 23 Jan 2021 05:18:26 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1227