----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128248/#review96754 -----------------------------------------------------------
I verified that the patch works, there's a boolean field called useSSL left in WikipediaApplet_p.h that should be removed, together with the MusicBrainz file. - Stefano Pettini On June 19, 2016, 11:44 a.m., Olivier Churlaud wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128248/ > ----------------------------------------------------------- > > (Updated June 19, 2016, 11:44 a.m.) > > > Review request for Amarok and Myriam Schweingruber. > > > Bugs: 349313 > https://bugs.kde.org/show_bug.cgi?id=349313 > > > Repository: amarok > > > Description > ------- > > I removed every possibility to use non SSL links > > > Diffs > ----- > > src/context/applets/wikipedia/WikipediaApplet.cpp 2ceb2b0 > src/context/applets/wikipedia/wikipediaGeneralSettings.ui 84cb5df > src/context/engines/wikipedia/WikipediaEngine.cpp 969d8fc > src/musicbrainz/MusicDNSAudioDecoder.cpp ea39a1b > > Diff: https://git.reviewboard.kde.org/r/128248/diff/ > > > Testing > ------- > > It compiles. However not tested, because make install always install in > /usr/local, whatever I give in -DCMAKE_INSTALL_PREFIX. And I don't want to > mess my install. > > > Thanks, > > Olivier Churlaud > >
