D15679: Add missing dependencies for PART_STAGE

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Mon Sep 24 12:16:55 BST 2018


leinir accepted this revision.
leinir added a comment.
This revision is now accepted and ready to land.


  > Maybe it would be nice to replace the Q_ASSERT with an if (factory) {} and put a warning in the else part in case of a wrong installation and plugins are missing. What do you think ?
  
  Something more informative than a straight up crash would, at any rate, be handy to have, yes... even an assert with a message would be better than just an outright assert :)

REPOSITORY
  R8 Calligra

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

To: dcaliste, leinir, danders, anthonyfieroni, #calligra:_3.0
Cc: Calligra-Devel-list, dcaliste, cochise, vandenoever
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20180924/c0d85698/attachment.html>


More information about the calligra-devel mailing list