Review Request 115774: Remove code that allows KStyle to build standalone

[David Faure]

> On Feb. 16, 2014, 11:40 a.m., Alex Merry wrote:
> > One thing worth considering as part of this is whether extra integration classes should get their own libraries or be put in the same one as KStyle.  If the latter, we probably want to rename KF5::Style.
> 
> David Faure wrote:
>     The kstyle lib is for KStyle (which only widget styles link to), so I think other integration bits, if any come up, will become different libs.
>     (unlikely, even, since libs that apps should link to, don't belong in tier4).
>     So I think KF5::Style is fine.
>     
>     
>     
>
> 
> Alex Merry wrote:
>     That, of course, means we're getting styles to find_package(KF5FrameworkIntegration) to use KF5::Style which ends up being a little surprising.  At least with KIO, it's fairly clear where KF5::KIOCore and KF5::KIOWidgets come from.
>     
>     I can sort of see the advantage of having a FrameworkIntegration config file, if only to allow for a RUNTIME dependency for workspace, but it would be nice to also allow find_package(KF5Style) like before.  Possibly by installing a KF5StyleConfig.cmake file as before, and having KF5FrameworkIntegration list that as a dep.

I think "couldn't find package frameworkintegration" is a clearer error message than "couldn't find package kstyle", when there is no kstyle framework.

That's from the user point of view.

>From the developer point of view, you're right, it's not obvious that you have to find_package(FrameworkIntegration) to get KF5::Style. But developers can dig a little bit more :-)

In any case my main goal is consistency with the other frameworks, for ease of maintainance, which also pushes towards find_package(frameworkintegration) as in this commit.


- David


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


On Feb. 16, 2014, 11:31 a.m., David Faure wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115774/
> -----------------------------------------------------------
> 
> (Updated Feb. 16, 2014, 11:31 a.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: frameworkintegration
> 
> 
> Description
> -------
> 
> Remove code that allows KStyle to build standalone
> 
> This is just excess maintenance burden for no real gain.
> 
> This commit includes RR 115700 by Michael Palimaka:
> guiaddons and itemviews doesn't appear to be used anywhere, so remove it.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 0d9551031037a0c33a5ea73a05151491d698235b 
>   src/kstyle/CMakeLists.txt f1379c852005c816dc80da4717ffb113d5a0d0a0 
>   src/kstyle/KF5StyleConfig.cmake.in  
> 
> Diff: https://git.reviewboard.kde.org/r/115774/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Faure
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140216/ccd1ca12/attachment.html>