D14288: Initial version of Clazy analyzer plugin

[Pino Toscano]

pino added a comment.


  In D14288#311683 <https://phabricator.kde.org/D14288#311683>, @antonanikin wrote:
  
  > > FWIW: Okular uses the Discount library for reading markdown documents, showing them as HTML. Also Cantor will do so too: D14738 <https://phabricator.kde.org/D14738>: Add the markdown entry.
  > > 
  > > Maybe it'd be a good idea to use it -- there are few options:
  > > 
  > > - make the library a hard dependency for this plugin (which then would not be built if discount is not found
  > > - optionally use discount, using this simple fallback otherwise (though with the risk of rotting)
  > > - optionally use discount, and show no documentation if not found
  >
  > I think about this library before writing current custom parser. It looks good and easy to use but adds extra dependency, especially for AppImage/Windows installer.
  
  
  Note that, unless you pack clazy as well, the plugin will be unusable for AppImage/Windows.

INLINE COMMENTS

> antonanikin wrote in test_clazyjob.cpp:82
> Sorry for not the exact formulation - clazy-standalone is runtime dependency for the plugin (real code analysis). This test will works without this executable - here we test only output and error parsing without executable calls. So should we skip the test when clazy-standalone is missing?

Clazy::Job creates a makefile that itself invokes clazy-standalone. Also, the outputs that are checked for (e.g. `stderrOutput1` & friends) are printed by clazy-standalone, aren't they?

REPOSITORY
  R32 KDevelop

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

To: antonanikin, #kdevelop
Cc: mwolff, apol, kfunk, brauch, pino, kossebau, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180820/b4c66189/attachment-0001.html>