----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125706/#review87090 -----------------------------------------------------------
>because kmix is a kdelibs4 application well, only when it's built that way :D KMIX_KF5_BUILD will enable KF5 codepath/buildsystem. IOW, you need to make sure both cases are covered - Hrvoje Senjan On Oct. 19, 2015, 6:30 p.m., Jonathan Riddell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125706/ > ----------------------------------------------------------- > > (Updated Oct. 19, 2015, 6:30 p.m.) > > > Review request for Plasma, David Edmundson and Harald Sitter. > > > Repository: plasma-pa > > > Description > ------- > > Install kconf_update script in kdelibs4 directory because kmix is a kdelibs4 > application and installing it in kf5 directories won't do anything > > This uses kde4-config. I looked at using KDELibs4 cmake classes from > KDELibs4/KDELibsDependencies.cmake but couldn't work out how to load it in > cmake and it has the wrong values set for its variables anyway. > > > Diffs > ----- > > data/CMakeLists.txt b652a0f > > Diff: https://git.reviewboard.kde.org/r/125706/diff/ > > > Testing > ------- > > Installs to the right place > > > Thanks, > > Jonathan Riddell > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel