> On June 13, 2014, 9:46 p.m., David Edmundson wrote: > > startkde/ksyncdbusenv/ksyncdbusenv.cpp, line 48 > > <https://git.reviewboard.kde.org/r/118730/diff/2/?file=281041#file281041line48> > > > > if we do .asyncCall() and waitForFinished() > > > > we may as well just do .call() > > > >
true and I tried and failed :-P (maybe it's just too late or too much football interrupt). call returns a QDBusMessage which doesn't have a isError(). - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118730/#review60042 ----------------------------------------------------------- On June 13, 2014, 9:36 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118730/ > ----------------------------------------------------------- > > (Updated June 13, 2014, 9:36 p.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
