Review Request 125029: Monitor timezone changing from timedated rather than our clock KCM

David Edmundson david at davidedmundson.co.uk
Wed Sep 2 14:53:54 UTC 2015



> On Sept. 2, 2015, 2:35 p.m., Martin Klapetek wrote:
> > But the ktimezoned should do exactly that; it monitors various files which timedate modifies and then sends that signal. Does ktimezoned not work in that case (it did when I last test it)? I think better place for this signal watcher would be in ktimezoned itself actually.
> 
> Martin Klapetek wrote:
>     Edit: timedatectl set-timezone Europe/Prague changes my clock immediatelly, so does "echo "Europe/Prague" > /etc/timezone

you are correct.

I can kill one of the two signals because of my other patch, but I'll merged that into there.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125029/#review84766
-----------------------------------------------------------


On Sept. 2, 2015, 10:33 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125029/
> -----------------------------------------------------------
> 
> (Updated Sept. 2, 2015, 10:33 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This means we get update the UI immedidately if the timezone is changed
> from an external source, such as timedatectl or some user script based
> on location for example.
> 
> This patch would put more of a dependency on timedated..unless we
> monitor both?
> 
> CCBUG: 351008
> 
> 
> Diffs
> -----
> 
>   dataengines/time/timeengine.cpp 2fd9792209ff5e78bd3dee1ed938eb0b1173de8d 
> 
> Diff: https://git.reviewboard.kde.org/r/125029/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150902/35db8986/attachment.html>


More information about the Plasma-devel mailing list