Using the search location getter added in D19338, ensure we get all locations which KNSCore::Engine will search for knsrc files, without making assumptions about where that might be.
Discover should probably not fallback, i.e. pass false as an argument.
This is Plasma 5.16 and by then we should have all applications within KDE ported. This can be an incentive for the rest to install in the right place.
Hmm! Very good point indeed, we're far enough away for this to be a thing we can sort for the majority of things, and yup, definitely avoids a very great amount of headaches :) i shall swap that and update the patch shortly
Needs to be 57 now (we'll get all of this landed soon, not to worry!)
Also, should the second one be KNEWSTUFFCORE_VERSION_MINOR? Seems weird that both conditions are checking the major version.
We'll need to wait until the next time the CI system gets updated for dependencies, otherwise it'll start being all red and angry - same for this one, incidentally, as what's there now will already report 5.57, but of course won't actually have the function being called... i've got patches under way for it all, though, but as we have to wait for the CI anyway, i've not stuck them up yet :)
All right, since the app dependency freeze is passed, it looks like any changes to apps will need to go into 19.08.0. Changes to anything on the Plasma release cycle can go in immediately since the next Plasma release will depend on Frameworks 5.58.