Review Request 125610: use org.kde.kuiserver5 dbus service (instead of org.kde.kuiserver)

[Rex Dieter]

> On Oct. 12, 2015, 5:11 p.m., Hrvoje Senjan wrote:
> > quick search on lxr.kde.org shows there are more uses of that interface, so changing the name now won't be easy (or recommended IMO)

OK, I'll revoke this request, and got with dropping kuiserver bits from kde-runtime distro/deployments (that use plasma5) for now.


- Rex


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


On Oct. 12, 2015, 4:53 p.m., Rex Dieter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125610/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2015, 4:53 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: 351500
>     https://bugs.kde.org/show_bug.cgi?id=351500
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Exclusively use org.kde.kuiserver5 dbus service so it doesn't conflict with kde4's org.kde.kuiserver from kde-runtime
> 
> 
> Diffs
> -----
> 
>   dataengines/applicationjobs/kuiserverengine.cpp 93ef2d6 
>   kuiserver/CMakeLists.txt 0f7c29e 
>   kuiserver/org.kde.kuiserver.service.in 7a86d07 
>   kuiserver/org.kde.kuiserver.xml 7d465bc 
>   kuiserver/org.kde.kuiserver5.service.in PRE-CREATION 
>   kuiserver/org.kde.kuiserver5.xml PRE-CREATION 
>   kuiserver/progresslistmodel.h 490a683 
>   kuiserver/progresslistmodel.cpp fe147f3 
> 
> Diff: https://git.reviewboard.kde.org/r/125610/diff/
> 
> 
> Testing
> -------
> 
> Applied patch, confirmed kuiserver builds, and plasma still functions as expected (it runs on plasma session login).  Admittedly, I'm not privy to kuiserver's detailed purpose, so wasn't sure how to test it specifically.
> 
> 
> Thanks,
> 
> Rex Dieter
> 
>

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