Opened 10 years ago

Closed 10 years ago

#6277 closed bug (fixed)

Deskbar doesn't update time settings

Reported by: darkwyrm Owned by: zooey
Priority: normal Milestone: R1
Component: Applications/Deskbar Version: R1/Development
Keywords: Cc: mdisreali@…
Blocked By: Blocking:
Platform: All


If you change the time via the Time prefs app, the Deskbar still shows the time using the old time zone.

Time zone change was from Greenwich to US/Eastern in hrev37415 GCC4 Hybrid.

Change History (8)

comment:1 by anevilyak, 10 years ago

Owner: changed from axeld to pulkomandy
Status: newassigned

comment:2 by pulkomandy, 10 years ago

Using the Locale kit and ICU, I had to drop the call to localtime(). This means ICU will now handle the timezone stuff. Unfortunately, it uses C++ objects, and these will not automagically update when you change some setting. So we'll have to tell ICU when a change of timezone occurs, so it can update the default BCountry object.

The other problem is the time preflet isn't aware of ICU yet, and could use it a lot more instead of messing with the timezone data directly. So it is not easy to plug them together.

comment:3 by Disreali, 10 years ago

Why was there a need to have ICU control timezones? I find it confusing that the Time Preflet no longer does the job for which it is designed. I'm sure that many users will also be confused by this.

comment:4 by Disreali, 10 years ago

Cc: mdisreali@… added

comment:5 by stippi, 10 years ago

Well I am sure anything not working to expectations is a temporary work in progress problem.

comment:6 by pulkomandy, 10 years ago

Owner: changed from pulkomandy to zooey

The time preflet is working properly, but the posix layer below is failing to take the changes into account.

comment:7 by Disreali, 10 years ago

Version: R1/alpha2R1/Development

Ticket #6370 has a much more descriptive summary of the same issue. Ticket #6396 also seems to be a duplicate.

comment:8 by zooey, 10 years ago

Resolution: fixed
Status: assignedclosed

Should be fixed in hrev38158, but I recommend testing it with hrev38165.

Note: See TracTickets for help on using tickets.