D16301: Remove ComponentInstaller

2018-11-19 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:38098e96e3d8: Remove ComponentInstaller (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16301?vs=43874=45796 REVISION

D16301: Remove ComponentInstaller

2018-11-19 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D16301 To: broulik, #plasma, mart Cc: apol, bruns, kde-frameworks-devel, michaelh, ngraham

D16301: Remove ComponentInstaller

2018-10-19 Thread Aleix Pol Gonzalez
apol added a comment. FWIW my +1 is to removing it. The Appstream assessment is all right but fixing something nobody uses sounds like a waste of time. :P REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D16301 To: broulik, #plasma, mart Cc: apol,

D16301: Remove ComponentInstaller

2018-10-19 Thread Kai Uwe Broulik
broulik added a comment. > Method InstallPackages Thanks for the insight. In any case, the way the code "works" right now is probably not what we want. It has been unused for years and is unlikely to come back in that form, so we can rm it. If we were to support something like this we

D16301: Remove ComponentInstaller

2018-10-18 Thread Aleix Pol Gonzalez
apol added a comment. +1 REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D16301 To: broulik, #plasma, mart Cc: apol, bruns, kde-frameworks-devel, michaelh, ngraham

D16301: Remove ComponentInstaller

2018-10-18 Thread Stefan BrĂ¼ns
bruns added a comment. https://github.com/hughsie/PackageKit/blob/master/src/org.freedesktop.PackageKit.Transaction.xml Method InstallPackages This requires of course we already have a valid package name. For this we probably have to rely on appstream, but need to define an

D16301: Remove ComponentInstaller

2018-10-18 Thread Kai Uwe Broulik
broulik created this revision. broulik added reviewers: Plasma, mart. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. broulik requested review of this revision. REVISION SUMMARY When a dataengine failed to load it would ask PackageKit to install it but it's