Review Request 118763: Remove find_package(XCB) call as it is handled already by the top-level cmake file

Bernd Steinhauser linux at bernd-steinhauser.de
Mon Aug 18 19:06:44 BST 2014



> On June 26, 2014, 11:49 a.m., Hugo Pereira Da Costa wrote:
> > Ship It!
> 
> Hugo Pereira Da Costa wrote:
>     Ping ? Should I commit this myself ?

Yes, please commit this. I don't have commit access myself.


- Bernd


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


On June 16, 2014, 2:07 p.m., Bernd Steinhauser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118763/
> -----------------------------------------------------------
> 
> (Updated June 16, 2014, 2:07 p.m.)
> 
> 
> Review request for kde-workspace, Plasma and Hugo Pereira Da Costa.
> 
> 
> Repository: oxygen
> 
> 
> Description
> -------
> 
> No idea if kde-workspace is still the right group, if not, please change.
> 
> find_package(XCB) is called without specifying the required components. This leads to linking to unused dependencies in case they are installed.
> 
> Since XCB is searched for in the top level cmake file in the repository, there is no need to search for it again. The component required there (only base XCB) is sufficient.
> Although, this should be sufficient to fix the deps problem, it makes sense to link to XCB::XCB instead of ${XCB_LIBRARIES}, since the former is what is actually needed.
> 
> 
> Diffs
> -----
> 
>   kstyle/CMakeLists.txt 165b62a 
>   liboxygen/CMakeLists.txt 0d1dd94 
> 
> Diff: https://git.reviewboard.kde.org/r/118763/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bernd Steinhauser
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140818/8106a92f/attachment.htm>


More information about the kde-core-devel mailing list