Review Request 129298: KPackage dependencies support

Aleix Pol Gonzalez aleixpol at kde.org
Thu Nov 17 16:00:33 UTC 2016


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

(Updated Nov. 17, 2016, 5 p.m.)


Review request for KDE Frameworks, Plasma, Dan Leinir Turthra Jensen, and Marco Martin.


Changes
-------

Full testing round.


Summary (updated)
-----------------

KPackage dependencies support


Repository: kpackage


Description (updated)
-------

Makes it possible to specify components to be installed together with a KPackage. They will be specified by a url, we'll have handlers for any kind, making reasonably extensible and doesn't tie us to a technology.

See RR in frameworksintegration for kns and appstream&packagekit: https://git.reviewboard.kde.org/r/129419/


Diffs (updated)
-----

  autotests/CMakeLists.txt 395b16e 
  autotests/data/testpackagesdep/contents/ui/main.qml PRE-CREATION 
  autotests/data/testpackagesdep/metadata.json PRE-CREATION 
  autotests/data/testpackagesdep/testpackagesdep.testappdataxml PRE-CREATION 
  src/kpackage/config-package.h.cmake 61f2f0c 
  src/kpackage/private/packagejobthread.cpp 0a0cc01 
  src/kpackage/private/packagejobthread_p.h 1babf0b 

Diff: https://git.reviewboard.kde.org/r/129298/diff/


Testing
-------

None. just builds. It's a proof of concept, not even the test I added was tested, it was just to display what it could look like.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161117/a2b900ba/attachment.html>


More information about the Plasma-devel mailing list