<table><tr><td style="">romangg 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/D10816">View Revision</a></tr></table><br /><div><div><p>Well we need to stop exporting the QT_QPA_PLATFORM value in startup script in any case, right? Or when we depend on Qt 5.11 and then set it to "wayland;x11" does it not break apps depending on Qt 5.y, y<11? So the question is just if we want to wait till we depend on 5.11 with this removal or do it now (and using the helper function proposed in this diff instead).</p>

<p>Putting this helper function into some framework is unnecessary though, that I can see now.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>platform-detection</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10816">https://phabricator.kde.org/D10816</a></div></div><br /><div><strong>To: </strong>graesslin, Plasma, romangg<br /><strong>Cc: </strong>davidedmundson, romangg, broulik, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>