Review Request 113643: Fix Standalone Build for KJobWidgets

Aurélien Gâteau agateau at kde.org
Tue Nov 5 14:29:07 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113643/#review43065
-----------------------------------------------------------



tier2/kjobwidgets/CMakeLists.txt
<http://git.reviewboard.kde.org/r/113643/#comment31061>

    While we are at it, one thing I do is factorize the framework version number like in template/CMakeLists.txt, that is:
    
    1. Add:
    
        set(KF5_VERSION "5.0.0")
    
    2. Replace uses of 5.0.0 in find_packages() and in ecm_setup_version() with ${KF5_VERSION}.


- Aurélien Gâteau


On Nov. 5, 2013, 3:18 p.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113643/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2013, 3:18 p.m.)
> 
> 
> Review request for KDE Frameworks and Aurélien Gâteau.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Specify required find_packages when not in kdelib's source dir.
> 
> 
> Diffs
> -----
> 
>   tier2/kjobwidgets/CMakeLists.txt 98bd0a1 
> 
> Diff: http://git.reviewboard.kde.org/r/113643/diff/
> 
> 
> Testing
> -------
> 
> 1. Configure a standalone build
> 
> Before this patch the configuration would complain of missing find_packages for KCore
> 
> 
> Thanks,
> 
> David Narváez
> 
>

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


More information about the Kde-frameworks-devel mailing list