D5447: Only add auto-detected compilers to model if they actually exist

[Milian Wolff]

mwolff added a comment.


  In D5447#440922 <https://phabricator.kde.org/D5447#440922>, @arrowd wrote:
  
  > I started working on it.
  >
  > I added method declaration to `kdevplatform/interfaces/iruntime.h` and wanted to add a default implementation in `iruntime.cpp`, but it turned out that to use `KDevelop::Path`, I have to include `<util/path.h>`, which belongs to `KDevPlatformUtil` library. I take it, `KDevPlatformInterfaces` isn't allowed to link to `KDevPlatformUtil`? How should I proceed?
  
  
  Don't supply a default implementation, interfaces should be pure (we have some default implementations there, but really that shouldn't be the case)

REPOSITORY
  R32 KDevelop

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

To: arrowd, mwolff, arichardson
Cc: skalinichev, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190401/f94eca3b/attachment.html>