<table><tr><td style="">kossebau created this revision.<br />kossebau added reviewers: Frameworks, aacid.<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/D23551">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 />
More, KXmlGui uses private API with Qt5::CorePrivate, so also future<br />
versions of Qt5::Core might not be compatible.</p>

<p>While the current code of KXmlGui 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.<br />
And given the use of private Qt::Core API, requiring exactly the same<br />
version by using "find_dependency(Qt5Core <span class="phabricator-remarkup-mention-unknown">@USED_QT_VERSION</span>@ EXACT)"<br />
should be more safe, even if the private API might be stable in reality<br />
for a set of versions.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Generated CMake config files now uses with the find_dependency(Qt5) calls<br />
the version of the Qt KXmlGui was built against.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</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/D23551">https://phabricator.kde.org/D23551</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
KF5XmlGuiConfig.cmake.in</div></div></div><br /><div><strong>To: </strong>kossebau, Frameworks, aacid<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>