Skip to content
  • Alexey Min's avatar
    Fix several build system errors · 731b8500
    Alexey Min authored
    Summary:
    This patch fixes two different, but closely related buil system errors.
    
    First one is about making KF5IconThemes and KF5ConfigWidgets really optional as intended. Without this patch, on a system with only Qt5 and other Tier 1 framework libs installed, you will get thw following cmake configure error:
    -- The following REQUIRED packages have not been found:
     * KF5 (required version >= 5.62.0)
    (full build logs below)
    
    Second error is revealed after the first one is fixed: plugin/kquickstyleitem.cpp includes ksharedconfig.h, but it cannot be found, because KConfig was not specified as dependency.
    
    qqc2-desktop-style is supposed to be Tier 1 framework, however it depends on several other Tier 1 frameworks (which is forbidden for Tier 1 frameworks?) and optionally depends on some Tier 3 frameworks, so shouldn't it be at least Tier 2 framework? Or even higher?
    Quote from https://api.kde.org/frameworks/index.html : "Tier 1 frameworks depend only on Qt (and possibly a small number of other third-party libraries), so can easily be used by any Qt-based project." - this is not true for qqc2-desktop-style.
    
    Test Plan: without this patch, try to build with only Qt5 and Tier 1 frameworks installed and see it fails
    
    Reviewers: mart, #plasma, dfaure
    
    Reviewed By: mart, #plasma
    
    Subscribers: cgiboudeaux, kde-frameworks-devel, plasma-devel
    
    Tags: #plasma
    
    Differential Revision: https://phabricator.kde.org/D24363
    731b8500