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


I'm happy with this. I think this should go in, even if there isn't another 
release, because packagers can pick it up as an "official" patch if they want. 
But I'll let Ivan have the final say.

- Alex Merry


On May 31, 2014, 3:42 a.m., Matthew Dawson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118340/
> -----------------------------------------------------------
> 
> (Updated May 31, 2014, 3:42 a.m.)
> 
> 
> Review request for KDE Frameworks, Plasma and Ivan Čukić.
> 
> 
> Repository: kactivities
> 
> 
> Description
> -------
> 
> To allow for the library to be co-installed with the frameworks version, 
> allow the daemon to be disabled.
> 
> I'm not sure if this is the best way to do this.  If there is any better way, 
> I'll update the patch as appropriate.
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt b4733648fd53ebd681694f185cc5b23f51dfd3f9 
> 
> Diff: https://git.reviewboard.kde.org/r/118340/diff/
> 
> 
> Testing
> -------
> 
> Visually inspected install directories.  Seems to remove only what is 
> necessary.
> 
> 
> Thanks,
> 
> Matthew Dawson
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to