> On June 21, 2016, 12:42 a.m., Olivier Churlaud wrote: > > Ship It!
It should work with new version: it's a must. If people complain about older version being broken, then we can see. But archlinux won't be able to ship Amarok without this fix. - Olivier ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126682/#review96762 ----------------------------------------------------------- On Jan. 9, 2016, 7:24 a.m., Diane Trout wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126682/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2016, 7:24 a.m.) > > > Review request for Amarok. > > > Repository: amarok > > > Description > ------- > > See API change documentation: > https://anonscm.debian.org/cgit/collab-maint/ffmpeg.git/tree/doc/APIchanges > https://wiki.libav.org/Migration/12 > > > Diffs > ----- > > src/musicbrainz/MusicDNSAudioDecoder.cpp > 6216e5ce26cc3461b89aea560943ad7a79101af9 > > Diff: https://git.reviewboard.kde.org/r/126682/diff/ > > > Testing > ------- > > I installed the patch on the debian 2.8.0 branch and it build successfully. I > then tried to trigger the affected code, however it crashed because of > https://bugs.kde.org/show_bug.cgi?id=328359 > > > Thanks, > > Diane Trout > >
