dfaure added a comment.
Thanks for the detailed analysis. I'm ok with option A. Let's not complicate this by adding submodule granularity. I have submitted the removal of the "package" member variable separately, that one is obvious and almost unrelated. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D24979 To: dfaure, kossebau, mart, apol Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns