Hi Tanu and Martin, Sorry for the late reply. I just pushed a new series of patches to raop2-for-merge2 branch. https://github.com/hfujita/pulseaudio-raop2/commits/raop2-for-merge2 Nothing has really been changed from raop2-for-merge1. It's just for rebasing and cleaning up some unused patches. Perhaps it would be better for Martin to rebase your new patches onto this branch before reviewing. If these patches are kind of trivial, I can merge them if you send them to me. (However the "refactoring" patch sounds non-trivial for me...) Thanks, Hajime Martin Blanchard wrote: > Hello Tanu, > > On jeu., 2013-12-05 at 11:29 +0200, Tanu Kaskinen wrote: >> On Sat, 2013-09-07 at 11:34 -0500, 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. >>> >>> While there are still some remaining issues, and testing has not >>> been done thoroughly, we'd like to have a review so that we can >>> discuss some design/style issues as early as possible. >> >> This patch set is now the oldest one in our queue, so I plan to start >> reviewing it soon. Has the RAOP work moved on after these patches were >> submitted? If so, could you send an updated patch set? >> > > The work we are doing on the RAOP module is on github, in a repo managed > by Hajime [1]. And yes, the code did move on after the patches send > here... > > Moreover, there are a few commits that I haven't submitted yet to > Hajime. Basically it brings an implementation for password > authentication, plus a large re-factoring that aims at merging old TCP > code path into the new UDP version. As of now, there are quiet some > duplication between the two implementations. > > I don't know if Hajime has time to build a new set of patches. On my > side, I need a few more weeks to finish re-factoring and testing the TCP > code. So it's probably better to wait for a new version I guess, or, at > least, for Hajime opinion on that. Anyway, thanks for the interest in > our work ! > > [1] https://github.com/hfujita/pulseaudio-raop2/commits/raop2-for-merge1 > > Martin > > > > _______________________________________________ > pulseaudio-discuss mailing list > pulseaudio-discuss at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss >
