[Differential] [Accepted] D4320: Don't fail when moving to the temp directory when installing

Marco Martin noreply at phabricator.kde.org
Sat Jan 28 17:02:20 UTC 2017


mart accepted this revision.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> installation.cpp:546
>              if (success) {
> +                if (installpath.startsWith(QDir::tempPath()))
> +                    file.remove(installpath);

if () {

}

also, add a comment on what is doing here

REPOSITORY
  R304 KNewStuff

BRANCH
  master

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

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

To: apol, #frameworks, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170128/0d6ff3af/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list