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

Gleb Popov noreply at phabricator.kde.org
Thu Jan 3 14:26:26 GMT 2019


arrowd updated this revision to Diff 48598.
arrowd added a comment.


  Pass absolute path instead of the compiler name to `createCompiler()`. Remove strange check that was breaking tests.
  
        
  
  Original patch by @arichardson.
  
  Differential Revision: D5447 <https://phabricator.kde.org/D5447>

REPOSITORY
  R32 KDevelop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D5447?vs=13429&id=48598

BRANCH
  d5447

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

AFFECTED FILES
  plugins/custom-definesandincludes/compilerprovider/compilerfactories.cpp
  plugins/custom-definesandincludes/compilerprovider/compilerprovider.cpp

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


More information about the KDevelop-devel mailing list