<table><tr><td style="">dfaure added a comment.
</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/D23815">View Revision</a></tr></table><br /><div><div><p>I'm going to adopt a different strategy for Frameworks after all, because:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">setting this in KDEFrameworksCompilerSettings.cmake would affect much more than frameworks (this stricter set of flags has been adopted by many many applications)</li>
<li class="remarkup-list-item">setting this in kdesrc-build's kf5-frameworks-build-include for all of frameworks would also affect deprecated modules like kdelibs4support which I'm too lazy to port since they'll disappear</li>
</ul>

<p>So what I'm going to do instead is set it in the toplevel CMakeLists.txt of each framework, after writing a script to do that.<br />
And instead of setting it to 0x060000 (which indeed breaks compilation whenever Qt changes, until 6.0) I'll set it to 5.13 (0x050d00) for now, then 5.14 (0x050e00) once 5.14 is released (not before, they might still deprecate more methods by then), and so on. If I'm the one doing the porting, I might as well trigger the breakage only when I have time to fix it.</p>

<p>Laurent, I think you should do the same in KDEPIM. If you want others to help, create a phabricator task (like we did with <a href="https://phabricator.kde.org/T11553" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T11553</a>). At least the one doing the porting will have decided to do that, rather than being forced to do it because he's the first one to upgrade Qt :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23815">https://phabricator.kde.org/D23815</a></div></div><br /><div><strong>To: </strong>dfaure, mdawson, arichardson, vkrause<br /><strong>Cc: </strong>pino, arojas, mlaurent, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>