ngraham added a comment.

  In D26544#591418 <https://phabricator.kde.org/D26544#591418>, @leinir wrote:
  
  > Right, the functionality would all still exist (that is, 
install/uninstall/update/whatnot would function just fine), what would happen 
without the dependent patch is that the views will be out of sync with the 
system state (basically the onChangedEntriesChanged will just fail to call the 
function it's pointed at due to incompatible types, in all but the desktoptheme 
kcm (which just calls load anyway, which doesn't expect anything weird).
  
  
  All right, that's probably not a hard blocker, especially since this was 
already broken. So we can land this patch for 5.18 IMO.

REPOSITORY
  R119 Plasma Desktop

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

To: leinir, mart
Cc: bcooksley, davidedmundson, ngraham, plasma-devel, LeGast00n, 
The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, 
alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, ahiemstra, mart

Reply via email to