[Differential] [Commented On] D3707: add simple test rig for service runner

sitter (Harald Sitter) noreply at phabricator.kde.org
Mon Dec 19 10:46:14 UTC 2016


sitter added inline comments.

INLINE COMMENTS

> dfaure wrote in servicerunnertest.cpp:5
> Is LGPL-v2-only on purpose here?

Not particularly. It's the same as the existing source file though, to have the same license restrictions apply to both. Is it worth changing it to gpgl2+kdev considering that?

> dfaure wrote in servicerunnertest.cpp:38
> Why futile? What is futile is the fear of breaking the user's cache ;)
> 
> You don't need any of this.
> 
> Since my big ksycoca refactoring a year ago, ksycoca's filename includes a SHA1 of the data dirs (see KSycoca::absoluteFilePath(), see also the filenames in your ~/.cache).
> So QStandardPaths::setTestModeEnabled(true); is already enough, given that it changes the local data home dir, so it changes the sha1.
> 
> See also e.g. kservice/autotests/kservicetest.cpp which doesn't change any env vars.
> 
> As a benefit it's also more portable to non-xdg platforms.

Awesome. Thanks!

REPOSITORY
  R120 Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sitter, broulik, dfaure
Cc: dfaure, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161219/d17f133f/attachment-0001.html>


More information about the Plasma-devel mailing list