Review Request 127448: After installing a package, load it
Marco Martin
notmart at gmail.com
Mon Mar 21 16:20:42 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127448/
-----------------------------------------------------------
(Updated March 21, 2016, 4:20 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Plasma.
Repository: kpackage
Description
-------
right now, to install a package from a tarball, you create an invalid package instance and then use its install method.
we don't know for sure what will be its final installation path, and is not easy to know it (one can blindly connect to the private PackageJob::installPathChanged, but that's an hack)
With this patch, the invalid instance used as installer, will load the package itself when the install job is finished, so if you connect to its finished signal, you are guaranteed the package instance is directly usable after that (provided the job didn't have any fatal errors)
Diffs
-----
autotests/plasmoidpackagetest.cpp eeaa0a3
src/kpackage/package.h 4ff4c70
src/kpackage/packagestructure.cpp accc5d1
src/kpackage/private/packagejob.cpp 76614a0
src/kpackage/private/packagejob_p.h acafa5e
Diff: https://git.reviewboard.kde.org/r/127448/diff/
Testing
-------
autotest included
Thanks,
Marco Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160321/36405022/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list