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.htm>
More information about the calligra-devel
mailing list