> On June 14, 2014, 9:12 p.m., Hrvoje Senjan wrote: > > startkde/ksyncdbusenv/CMakeLists.txt, line 5 > > <https://git.reviewboard.kde.org/r/118730/diff/2/?file=281040#file281040line5> > > > > it was my understanding that KF5_LIBEXEC_INSTALL_DIR is 'reserved' for > > KF5, and rest of depending software would use libexec_install_dir. > > iow, this is only executable in kf5/ subdir, that doesn't come from a > > Framework > > Martin Gräßlin wrote: > thanks for noticing, I'll fix it later today.
addressed in http://commits.kde.org/plasma-workspace/2e4ec651a5a12e8d46d1afd17b614aa290306ee0 - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118730/#review60095 ----------------------------------------------------------- On June 14, 2014, 10:41 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118730/ > ----------------------------------------------------------- > > (Updated June 14, 2014, 10:41 a.m.) > > > Review request for Plasma and David Edmundson. > > > Repository: plasma-workspace > > > Description > ------- > > [startkde] Introduce a ksyncdbusenv helper application > > This application is invoked by startkde once all environment variables > are set and syncs them to DBus for the activation environment. > > > Diffs > ----- > > startkde/startkde.cmake 0c1f5841eeefc1012dbdc71e1d1fde11e7511dd6 > startkde/ksyncdbusenv/ksyncdbusenv.cpp PRE-CREATION > startkde/ksyncdbusenv/CMakeLists.txt PRE-CREATION > startkde/CMakeLists.txt 3360a1c59409588ecec49e63276914b69568164e > > Diff: https://git.reviewboard.kde.org/r/118730/diff/ > > > Testing > ------- > > on log in kglobalacceld5 doesn't pick up the right environment yet, but on > manual invocation of the tool + trigger restart of kglobalacceld5 it works. > So the approach seems to work, just not yet from startkde (maybe it's too > late?) > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
