> On Feb. 9, 2015, 7:47 a.m., Martin Gräßlin wrote: > > kcms/dateandtime/main.cpp, line 57 > > <https://git.reviewboard.kde.org/r/122400/diff/4/?file=347911#file347911line57> > > > > does it need to be a blocking dbus call? Any chance in getting it use a > > watcher?
It's tricky as load() is called by systemsettings externally, and I need to have loaded the UI before that happens... and I need to know the result of this in order to load the UI... - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122400/#review75660 ----------------------------------------------------------- On Feb. 8, 2015, 5:18 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122400/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2015, 5:18 p.m.) > > > Review request for Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > The current time setting helper is incredibly broken. > > It manually tries to run a range of NTP utilities, all of which are > deprecated. > > We can just call timedated directly and cut out the middleman as it has > uses polkit anyway. > > This is currently an optional dependency, and the original helper still > exists. It makes the code messy, but we have users to support for now. > > Finding timedated is an cmake option rather than querying for systemd > libs to make it easier for those deploying shims, such as BSD. > > > (code is in two commits, first abstracting the saving from the dtime class; > then adding in the second save mechanism) > > > Diffs > ----- > > kcms/dateandtime/timedated1.xml PRE-CREATION > kcms/dateandtime/main.h c1e5234 > kcms/dateandtime/main.cpp 0041a9d > kcms/dateandtime/dtime.h 1a90698 > kcms/dateandtime/dtime.cpp 482e483 > kcms/dateandtime/CMakeLists.txt 4a987ae > kcms/dateandtime/dateandtime.ui c073b5e > > Diff: https://git.reviewboard.kde.org/r/122400/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel