D6104: Use the right scope for the installpath variable

[Aleix Pol Gonzalez]

apol created this revision.
Restricted Application added a project: Frameworks.

REVISION SUMMARY
  Only mark package as installed if it was successfully extracted
  
  Don't store all installed files in the knsregistry file
  
  At the moment we were storing every file, for big packages (for example
  icon themes like Papyrus) this meant ~5 MiB of paths that had to be loaded
  when the engine was created.
  This patch makes it so instead of adding all the contents of the directory
  it will only list the extracted directories so they can recursively be
  removed. I don't think it's a supported workflow to be adding stuff in an
  extracted directory anyway.

TEST PLAN
  Tests pass, tried un/installing massive icon themes. Also massif
  reports a much lower memory consumption.

REPOSITORY
  R304 KNewStuff

BRANCH
  installedFiles2

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

AFFECTED FILES
  src/core/installation.cpp

To: apol, #frameworks, whiting, leinir
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170605/552148f1/attachment.html>