Review Request 129237: [amarok] Try port services to KF5

Anthony Fieroni bvbfan at abv.bg
Sat Nov 26 06:33:39 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129237/#review101123
-----------------------------------------------------------



The problem with application exist is 3 active threads to MyQSL keeps alive, mysql_thread_end() nor mysql_library_end() are never called, QThreadStorage does not destroy objects. I don't havve much time for this and i can discard if it does not help you.

- Anthony Fieroni


On Окт. 21, 2016, 5:01 след обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129237/
> -----------------------------------------------------------
> 
> (Updated Окт. 21, 2016, 5:01 след обяд)
> 
> 
> Review request for Amarok.
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> I try to enable services in KF5 branch, amarok to be usable, but problems still present
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt e777540 
>   src/MainWindow.cpp c8ca2fe 
>   src/main.cpp c7e235b 
>   src/services/ampache/AmpacheSettings.cpp 0056d02 
>   src/services/gpodder/GpodderServiceSettings.cpp 34280c7 
>   src/services/jamendo/JamendoService.cpp 709f668 
>   src/services/lastfm/LastFmServiceSettings.cpp 4a79ce6 
>   src/services/magnatune/MagnatuneCollectionLocation.cpp 4ec21e7 
>   src/services/magnatune/MagnatuneDownloadDialog.cpp 05847a3 
>   src/services/magnatune/MagnatuneInfoParser.cpp 143b942 
>   src/services/magnatune/MagnatuneSettingsModule.cpp 702a16a 
>   src/services/magnatune/MagnatuneStore.cpp 3441e40 
>   src/services/mp3tunes/Mp3tunesSettingsModule.cpp 5cf5999 
>   src/services/opmldirectory/OpmlDirectoryInfoParser.cpp ea44956 
>   src/services/opmldirectory/OpmlDirectoryModel.cpp d58e17a 
>   src/services/opmldirectory/OpmlDirectoryService.cpp 3f97bbf 
> 
> Diff: https://git.reviewboard.kde.org/r/129237/diff/
> 
> 
> Testing
> -------
> 
> "amarok_storage-mysqlserverstorage.desktop" has no desktop group, cannot construct a KPluginInfo object from it.
> "amarok_storage-mysqlestorage.desktop" has no desktop group, cannot construct a KPluginInfo object from it.
> "amarok_collection-mysqlcollection.desktop" has no desktop group, cannot construct a KPluginInfo object from it.
> 
> [Desktop Entry] presents, but still not working.
> 
> One more issue, application deadlocks at exit.
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20161126/4ccd20d0/attachment.html>


More information about the Amarok-devel mailing list