D9018: Don't cause circular linking on Windows

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Mon Nov 27 14:41:52 UTC 2017


leinir created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.

REVISION SUMMARY
  When building plugins, don't arbitrarily rename the output files (as this will occasionally result in circular dependencies).
  
  In this review from three and a half years ago <https://git.reviewboard.kde.org/r/118084/>, 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.

TEST PLAN
  With these properties set, the plugins fail to load on Windows due to circular dependencies
  
  Without them (that is, with this patch), the plugins load

REPOSITORY
  R246 Sonnet

REVISION DETAIL
  https://phabricator.kde.org/D9018

AFFECTED FILES
  src/plugins/aspell/CMakeLists.txt
  src/plugins/enchant/CMakeLists.txt
  src/plugins/hspell/CMakeLists.txt
  src/plugins/hunspell/CMakeLists.txt
  src/plugins/nsspellchecker/CMakeLists.txt
  src/plugins/voikko/CMakeLists.txt

To: leinir
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171127/d851bada/attachment.html>


More information about the Kde-frameworks-devel mailing list