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.
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.
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.
Finally got that song out of my head.......
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.
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1227