Review Request 130142: AAC support was broken due to ffmpeg deprecating the currently-used libfaac transcoder. Reworked to test for the new one.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130142/ --- Review request for Amarok. Repository: amarok Description --- Working AAC format detection and transcoding option. I have to admit that most of this work was done by a helpful user on the Ubuntu forums, but I have tested the patch. See https://ubuntuforums.org/showthread.php?t=2348624=2=13642704#post13642704 for the original discovery of the bug. It is also on the Amarok bug list at https://bugs.kde.org/show_bug.cgi?id=374670#c11 Diffs - src/core/transcoding/formats/TranscodingAacFormat.cpp 7dcd4cf Diff: https://git.reviewboard.kde.org/r/130142/diff/ Testing --- I patched Amarok 2.8.4 and then compiled Amarok. I then tested to see if AAC transcoding worked. The option appeared and it did indeed transcode to AAC format. I also updated the patch to work with the latest version from Git here at KDE (however the patch had been created, it was not in git format). Thanks, Russell Andres
Re: Review Request 130142: AAC support was broken due to ffmpeg deprecating the currently-used libfaac transcoder. Reworked to test for the new one.
> On May 30, 2017, 8:39 a.m., Myriam Schweingruber wrote: > > Hm, what is Amarok 2.8.4? That is not a release version number I am familar > > with. > > > > Are you the same person who comments on the bug? Did you test this patch, > > e.g. does Amarok build and does it pass all the unit tests? Yes, I am. I installed the patch on the Ubuntu package source. I was having trouble getting Amarok to build (specifically, utter failure), with or without the patch. Since the main part of the change is in the test for AAC transcoding capability, I decided to test it on the Ubuntu version, which is closely based on this one, but which had build instructions and an easy installation method built-in (since I could then just use the package manager to get dependencies and install). The tests seem to have worked, and Amarok certainly transcoded to AAC format. I am afraid that I can't probably help test further on the latest version until I can figure out what I need to install as build dependencies and what options I need to give it to compile, though. - Russell --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130142/#review103252 --- On May 26, 2017, 12:22 a.m., Russell Andres wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/130142/ > --- > > (Updated May 26, 2017, 12:22 a.m.) > > > Review request for Amarok. > > > Repository: amarok > > > Description > --- > > Working AAC format detection and transcoding option. I have to admit that > most of this work was done by a helpful user on the Ubuntu forums, but I have > tested the patch. See > https://ubuntuforums.org/showthread.php?t=2348624=2=13642704#post13642704 > for the original discovery of the bug. It is also on the Amarok bug list at > https://bugs.kde.org/show_bug.cgi?id=374670#c11 > > > Diffs > - > > src/core/transcoding/formats/TranscodingAacFormat.cpp 7dcd4cf > > Diff: https://git.reviewboard.kde.org/r/130142/diff/ > > > Testing > --- > > I patched Amarok 2.8.4 and then compiled Amarok. I then tested to see if AAC > transcoding worked. The option appeared and it did indeed transcode to AAC > format. I also updated the patch to work with the latest version from Git > here at KDE (however the patch had been created, it was not in git format). > > > Thanks, > > Russell Andres > >
Re: Review Request 130142: AAC support was broken due to ffmpeg deprecating the currently-used libfaac transcoder. Reworked to test for the new one.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130142/ --- (Updated Feb. 4, 2018, 10:26 p.m.) Status -- This change has been marked as submitted. Review request for Amarok. Changes --- Submitted with commit 83c7357f0712aeeccbf083bc9462865f6acecf9e by Stefano Pettini to branch master. Repository: amarok Description --- Working AAC format detection and transcoding option. I have to admit that most of this work was done by a helpful user on the Ubuntu forums, but I have tested the patch. See https://ubuntuforums.org/showthread.php?t=2348624=2=13642704#post13642704 for the original discovery of the bug. It is also on the Amarok bug list at https://bugs.kde.org/show_bug.cgi?id=374670#c11 Diffs - src/core/transcoding/formats/TranscodingAacFormat.cpp 7dcd4cf Diff: https://git.reviewboard.kde.org/r/130142/diff/1/ Testing --- I patched Amarok 2.8.4 and then compiled Amarok. I then tested to see if AAC transcoding worked. The option appeared and it did indeed transcode to AAC format. I also updated the patch to work with the latest version from Git here at KDE (however the patch had been created, it was not in git format). Thanks, Russell Andres