<table><tr><td style="">leinir created this revision.<br />Restricted Application added a project: Frameworks.<br />Restricted Application added a subscriber: Frameworks.
</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/D9018" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When building plugins, don't arbitrarily rename the output files (as this will occasionally result in circular dependencies).</p>

<p><a href="https://git.reviewboard.kde.org/r/118084/" class="remarkup-link" target="_blank" rel="noreferrer">In this review from three and a half years ago</a>, the Sonnet plugins were moved into a subdirectory, which was great. They were, however, also renamed at the same time, which resulted in some (and specifically the aspell plugin) being named the same as the library they should be linked against. End result: Spell checking has not worked on Windows for three and a half years.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>With these properties set, the plugins fail to load on Windows due to circular dependencies</p>

<p>Without them (that is, with this patch), the plugins load</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R246 Sonnet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9018" rel="noreferrer">https://phabricator.kde.org/D9018</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/plugins/aspell/CMakeLists.txt<br />
src/plugins/enchant/CMakeLists.txt<br />
src/plugins/hspell/CMakeLists.txt<br />
src/plugins/hunspell/CMakeLists.txt<br />
src/plugins/nsspellchecker/CMakeLists.txt<br />
src/plugins/voikko/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>leinir<br /><strong>Cc: </strong>Frameworks<br /></div>