<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/D8705" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Qt doesn't have files like KF5WindowSystemConfig.cmake.in  :)</p>

<p>I guess you mean changing #includes in public headers, but this is also about changing the library dependencies, which I have never seen change in Qt.</p>

<p>Now that you remind me that Qt sometimes changes includes in public headers, it reminds me that SC for this case is indeed debatable, if an app was using Foo it was supposed to include foo.h itself, rather than indirectly.</p>

<p>I guess that the same argument could be made about Qt5::Widgets... if you're using a widget you're supposed to link to that, rather than indirectly via KWindowSystem...</p>

<p>This is making me change my mind. Maybe it's OK after all. It breaks SC in a case where "proper" application code shouldn't be broken.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R278 KWindowSystem</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8705" rel="noreferrer">https://phabricator.kde.org/D8705</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, dfaure<br /><strong>Cc: </strong>graesslin, dfaure, anthonyfieroni<br /></div>