D16733: Capture package metadata before install start

2018-11-08 Thread José Manuel Santamaría Lema
joselema removed a commit: R242:ed1724c63e9f: Summary: After this commit https://cgit.kde.org/plasma-framework.git/commit/?. REPOSITORY R290 KPackage REVISION DETAIL https://phabricator.kde.org/D16733 To: davidedmundson, apol Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns

D16733: Capture package metadata before install start

2018-11-07 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R290:451b1785e8d0: Capture package metadata before install start (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D16733?vs=45026=45028#toc REPOSITORY R290 KPackage

D16733: Capture package metadata before install start

2018-11-07 Thread Aleix Pol Gonzalez
apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. I would add a comment saying "please don't call d->package from the lambda because it breaks". REPOSITORY R290 KPackage BRANCH master REVISION DETAIL https://phabricator.kde.org/D16733

D16733: Capture package metadata before install start

2018-11-07 Thread David Edmundson
davidedmundson created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY KPackage (despite being supposedly internally shared) is not kept as a strong reference inside