> On Lug. 6, 2014, 1:25 p.m., David Faure wrote: > > I added GenericConfigLocation to Qt 5.2.0's QStandardPaths, but indeed it's > > missing in qtpaths, I'll fix that. > > > > We should be using it, because ConfigLocation includes the appname on > > Windows (big whoops). > > David Faure wrote: > I mean we should be using it once we can depend on Qt 5.4 :) > For now ConfigLocation will do.
I'll add a TODO so that it is not forgotten. - Luca ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119145/#review61701 ----------------------------------------------------------- On Lug. 6, 2014, 1:19 p.m., Luca Beltrame wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119145/ > ----------------------------------------------------------- > > (Updated Lug. 6, 2014, 1:19 p.m.) > > > Review request for KDE Frameworks, Plasma and David Faure. > > > Bugs: 333793 > https://bugs.kde.org/show_bug.cgi?id=333793 > > > Repository: plasma-workspace > > > Description > ------- > > Currently env and shutdown scripts in startkde are not run due to the way > kf5-config is used. There is no more $KDEHOME, so proper paths aren't being > found. > > This patch moves things to qtpaths, putting things under ConfigLocation + > "plasma-workspace" + "env" (or "shutdown"). This is what the plasma-desktop > KCM does, but according to the discussion in [1] it may change depending on > the outcome. > > [1] http://lists.kde.org/?t=140464151200002&r=1&w=2 > > > Diffs > ----- > > startkde/startkde.cmake 24610f4 > > Diff: https://git.reviewboard.kde.org/r/119145/diff/ > > > Testing > ------- > > > Thanks, > > Luca Beltrame > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
