leinir updated this revision to Diff 80289. leinir added a comment.
Address @ngraham's (and my own) worry about the synchronous behaviour exhibited by KPackage... Something a bit like this probably wants to go into KPackage itself, perhaps we can consider this after we've done a bit of testing of its solidity here... - Add a simple async job wrapper for KPackage operations - Add the KPackage job to the build - Switch KNSCore::Installation to using the new async KPackageJob REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28701?vs=80195&id=80289 BRANCH add-kpackage-support (branched from master) REVISION DETAIL https://phabricator.kde.org/D28701 AFFECTED FILES CMakeLists.txt src/core/CMakeLists.txt src/core/cache.cpp src/core/cache.h src/core/engine.cpp src/core/engine.h src/core/installation.cpp src/core/installation.h src/core/jobs/kpackagejob.cpp src/core/jobs/kpackagejob.h src/downloaddialog.h To: leinir, #plasma, #knewstuff, #frameworks, ngraham, mart, davidedmundson Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns
