https://bugs.kde.org/show_bug.cgi?id=374766

Marco Martin <notm...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/405518c41da9df963823eed9c
                   |                            |7688b4353f29fe6

--- Comment #3 from Marco Martin <notm...@gmail.com> ---
Git commit 405518c41da9df963823eed9c7688b4353f29fe6 by Marco Martin.
Committed on 20/01/2017 at 14:34.
Pushed by mart into branch 'Plasma/5.9'.

use proper packagestructures

Summary:
as the plasmapkg2 utility is dead, scripts and tabboxes couldn't
be installed anymore, as plasmapkg2 was launched with an hardcoded
package type (and had a bunch of custom code for it)
and no packagestructure for the relevant package types existed.

the port of scripts and tabboxes to kpackage in kwin is quite
incomplete and somewhat strange (given the fact that was started by
looking manually for the file as it couldn't link to libplasma)

this makes things a bit better as now scripts and tabboxes have an actual
package structure, making it installable by kpackagetool5.
in the future, the port should be completed and replace all the manual lookup
of files with package::filePath as it should work now given that valid
structures exist

Test Plan:
tabbox plugins and kwin scripts can be installed from knewstuff
again and can be successfully loaded

Reviewers: #plasma, graesslin, subdiff

Reviewed By: #plasma, graesslin

Subscribers: graesslin, plasma-devel, kwin, #kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D4202

M  +1    -0    CMakeLists.txt
M  +8    -8    kcmkwin/kwintabbox/main.cpp
A  +2    -0    packageplugins/CMakeLists.txt
A  +16   -0    packageplugins/scripts/CMakeLists.txt
A  +10   -0    packageplugins/scripts/kwin-packagestructure-scripts.desktop
A  +62   -0    packageplugins/scripts/scripts.cpp     [License: LGPL (v2+)]
A  +33   -0    packageplugins/scripts/scripts.h     [License: LGPL (v2+)]
A  +16   -0    packageplugins/windowswitcher/CMakeLists.txt
A  +10   -0   
packageplugins/windowswitcher/kwin-packagestructure-windowswitcher.desktop
A  +62   -0    packageplugins/windowswitcher/windowswitcher.cpp     [License:
LGPL (v2+)]
A  +33   -0    packageplugins/windowswitcher/windowswitcher.h     [License:
LGPL (v2+)]

https://commits.kde.org/kwin/405518c41da9df963823eed9c7688b4353f29fe6

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to