Opened 9 years ago

Closed 9 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:
Has a Patch: no Platform: All

Description

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 Changed 9 years ago by anevilyak

Owner: changed from axeld to pulkomandy
Status: newassigned

comment:2 Changed 9 years ago by pulkomandy

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 Changed 9 years ago by Disreali

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 Changed 9 years ago by Disreali

Cc: mdisreali@… added

comment:5 Changed 9 years ago by stippi

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

comment:6 Changed 9 years ago by pulkomandy

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 Changed 9 years ago by Disreali

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 Changed 9 years ago by zooey

Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.