<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 actually think it's a mistake to set this define unconditionally in PIM.</p>

<p>If a user of the last PIM release (as in, a person or a distro) wants to use Qt 5.14, they'll get compilation errors, unnecessarily.</p>

<p>If we want to enable this flag (to "punish" the first KDE developer who upgrades Qt, like me currently), then at least we should only do so in git checkouts, not in release tarballs.<br />
This is easy to do: <tt style="background: #ebebeb; font-size: 13px;">if (EXISTS "${CMAKE_SOURCE_DIR}/.git")</tt>.<br />
I'm actually thinking of doing that (in KDEFrameworkCompilerSettings), once everything builds again (I obviously can't push that just yet).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</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<br /><strong>Cc: </strong>arojas, mlaurent, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>