asturmlechner added a comment.
In D19338#676682 <https://phabricator.kde.org/D19338#676682>, @kossebau wrote: > In D19338#676680 <https://phabricator.kde.org/D19338#676680>, @asturmlechner wrote: > > > Hm, why was this new variable KDE_INSTALL_KNSRCDIR not put into KDEInstallDirs? > > > On the other hand it makes sense that module-specific variables are provided by the CMake config file of the respective module. Which allows projects to use the module without having to use KDEInstallDirs, but e.g. GnuInstallDirs. That could be done with checking in KNewStuff if KDE_INSTALL_KNSRCDIR isn't already set, then provide a fallback value, right? Because right now it means just installing a .knsrc file into the right path requires KNewStuff. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D19338 To: leinir, #knewstuff, apol, ngraham, fvogt Cc: asturmlechner, kossebau, kde-frameworks-devel, #knewstuff, LeGast00n, cblack, michaelh, ZrenBot, ngraham, bruns
