Review Request 129753: search and replace old hardcoded types from plasmapkg2

Marco Martin notmart at gmail.com
Tue Jan 3 12:25:13 UTC 2017


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

(Updated Jan. 3, 2017, 12:25 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Aleix Pol Gonzalez.


Changes
-------

Submitted with commit 642db25fb9eef5913b592e5435fe3f4a6c8fc732 by Marco Martin to branch master.


Bugs: 374463
    http://bugs.kde.org/show_bug.cgi?id=374463


Repository: plasma-framework


Description
-------

there is a single difference between command line parameters of plasmapkg2 and kpackagetool5: plasmapkg2 had some hardcoded names like "theme", "plasmoid" etc.
kpackagetool5 requires the servicetype name to be passed instead. in order to stay compatible convert those parameter with the proper servicetype name.
it's parsing parameters by hand in order to just pass anything else without further parsing


Diffs
-----

  src/plasma/packagestructure/plasmathemepackage.cpp 94f0c5c 
  src/plasmapkg/main.cpp b61b2e5 

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


Testing
-------

installing themes from knewstuff works again


Thanks,

Marco Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170103/12194fe9/attachment-0001.html>


More information about the Plasma-devel mailing list