<table><tr><td style="">kossebau created this revision.<br />kossebau added a reviewer: Frameworks.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />kossebau requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23550">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Some developers have multiple versions of Qt installed on their system. <br />
And source code often has some version-dependent code paths, e.g. using<br />
newer API if found available. In that case the resulting build has an<br />
effective dependency on the Qt version built against.</p>

<p>To properly reflect that, the Qt version set as minimally required version<br />
of the listed dependencies in the generated CMake config files should use<br />
that version instead.<br />
While the current code of KCoreAddons does not have such version dependent<br />
code variants, they are added now and then, so using the Qt version built<br />
against as minimally required dependency is a good default, and should not<br />
change anything for all the users with just one Qt version, and improve<br />
things for those with, catching bad mixes early in the build time.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Generated CMake config files now uses with the find_dependency(Qt5) call the<br />
version of the Qt KCoreAddons was built against.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>BRANCH</strong><div><div>setusetqtversionasmindep</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23550">https://phabricator.kde.org/D23550</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
KF5CoreAddonsConfig.cmake.in</div></div></div><br /><div><strong>To: </strong>kossebau, Frameworks<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>