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