> On Aug. 25, 2014, 5:02 p.m., David Faure wrote: > > kde-modules/KDEInstallDirs.cmake, line 340 > > <https://git.reviewboard.kde.org/r/119901/diff/2/?file=307587#file307587line340> > > > > I'd name that ECM_INSTALL_PLUGINS_IN_QT_DIR to be clearer about what > > it's about. > > > > Is this line enough? I thought you had to use option() to make the > > option available on the cmake command-line, but maybe I'm missing something. > > Rohan Garg wrote: > I think ECM_INSTALL_PLUGINS_IN_QT_DIR would be wrong since it's used for > both plugins and QtQuick imports.
Yeah but as it is now, it sounds like EVERYTHING is going to be installed into the Qt dir :-) Calling it ECM_INSTALL_PLUGINS_AND_IMPORTS_IN_QT_DIR would be quite a mouthful. In fact, IMHO imports are kind of plugins too. That dir is indeed full of .so plugins... so yeah, I still prefer ECM_INSTALL_PLUGINS_IN_QT_DIR :) - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119901/#review65234 ----------------------------------------------------------- On Aug. 25, 2014, 5:17 p.m., Rohan Garg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119901/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2014, 5:17 p.m.) > > > Review request for Build System and KDE Frameworks. > > > Repository: extra-cmake-modules > > > Description > ------- > > Use qmake to query dirs for plugins and imports instead of hardcoding them in > ECM. > > > Diffs > ----- > > kde-modules/KDEInstallDirs.cmake 880539b > > Diff: https://git.reviewboard.kde.org/r/119901/diff/ > > > Testing > ------- > > Seems to work on my system. > > > Thanks, > > Rohan Garg > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
