D23550: CMake config files: use as min dep version the Qt version we built against

[Friedrich W. H. Kossebau]

kossebau created this revision.
kossebau added a reviewer: Frameworks.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  Some developers have multiple versions of Qt installed on their system. 
  And source code often has some version-dependent code paths, e.g. using
  newer API if found available. In that case the resulting build has an
  effective dependency on the Qt version built against.
  
  To properly reflect that, the Qt version set as minimally required version
  of the listed dependencies in the generated CMake config files should use
  that version instead.
  While the current code of KCoreAddons does not have such version dependent
  code variants, they are added now and then, so using the Qt version built
  against as minimally required dependency is a good default, and should not
  change anything for all the users with just one Qt version, and improve
  things for those with, catching bad mixes early in the build time.

TEST PLAN
  Generated CMake config files now uses with the find_dependency(Qt5) call the
  version of the Qt KCoreAddons was built against.

REPOSITORY
  R244 KCoreAddons

BRANCH
  setusetqtversionasmindep

REVISION DETAIL
  https://phabricator.kde.org/D23550

AFFECTED FILES
  CMakeLists.txt
  KF5CoreAddonsConfig.cmake.in

To: kossebau, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190828/26c98301/attachment.html>