[pulseaudio-discuss] [PATCH v7 00/33] raop2 support for module-raop-sink

[Tanu Kaskinen]

On Thu, 2017-01-05 at 22:34 +0100, Anton Lundin wrote:
> On 06 November, 2016 - Hajime Fujita wrote:
> 
> > This patch set adds a support for UDP version of RAOP (so called
> > raop2). Most of the RAOP devices (e.g. AppleTV, AirportExpress,
> > third party AV receivers) today use UDP version, so this patch
> > set is expected to support those devices.
> > 
> 
> I've took a close look at the non-raop changes and those LGTM.
> 
> Feel free to add a Reviewed-by: Anton Lundin <glance at acc.umu.se> to them
> if you feel for it.
> 
> I've tested and glanced at the raop-code and it ain't perfect but its
> way better than the current code for anything modern, so I'd suggest
> merging this.
> 
> Feel free to add a Tested-by: Anton Lundin <glance at acc.umu.se> to them
> if you feel for it.

Thanks, I pushed the three core-util patches to the "next" branch. I
started to apply the rest of the patches too, but the first raop patch
failed to build:

  CC       modules/raop/module_raop_sink_la-module-raop-sink.lo
modules/raop/module-raop-sink.c: In function ‘udp_thread_func’:
modules/raop/module-raop-sink.c:724:19: error: too many arguments to function ‘pa_rtpoll_run’
         if ((rv = pa_rtpoll_run(u->rtpoll, true)) < 0)
                   ^~~~~~~~~~~~~
In file included from ./pulsecore/source.h:41:0,
                 from ./pulsecore/core.h:49,
                 from ./pulsecore/sink.h:33,
                 from modules/raop/module-raop-sink.c:44:
./pulsecore/rtpoll.h:61:5: note: declared here
 int pa_rtpoll_run(pa_rtpoll *f);
     ^~~~~~~~~~~~~
Makefile:9124: recipe for target 'modules/raop/module_raop_sink_la-module-raop-sink.lo' failed

Maybe that's fixed in later patches, but for bisecting it's best to
avoid commits that break the build.

I also tried to build the whole patch set, but Debian has updated to
openssl 1.1.0, and at the time when these patches were submitted, we
didn't yet have the patch that fixes the compatibility issue with
openssl 1.1.0, so that failed too. Trying to apply both the openssl fix
and the raop patches results in conflicts.

These issues should be easy enough to resolve, and if there's nothing
more complicated, I'll fix the issues myself and push the patches.

-- 
Tanu

https://www.patreon.com/tanuk