<table><tr><td style="">bcooksley requested changes to this revision.<br />bcooksley 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/D26752">View Revision</a></tr></table><br /><div><div><p>Christophe is correct here, it is worth warning developers about these issues regardless of the platform, so they can get the code ready for those platforms and test everything in their local environment as much as possible.<br />
I know for certain that there are developers who rely on our CI system and the Binary Factory to test and validate their applications (because they themselves do not have access to a development environment on those platforms).</p>

<p>If this warning were to occur only on the platform(s) which it impacts then it would become much harder for people to fix and debug.</p>

<p>Of course, if you aren't targeting those platforms, you can just do the necessary if() to not hit this path.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26752">https://phabricator.kde.org/D26752</a></div></div><br /><div><strong>To: </strong>patrickelectric, apol, tcanabrava, cgiboudeaux, bcooksley<br /><strong>Cc: </strong>bcooksley, patrickelectric, apol, cgiboudeaux, kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, michaelh, ngraham, bruns<br /></div>