After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 677512 - Port clock applet to 2012
Port clock applet to 2012
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: clock
3.4.x
Other Linux
: Normal normal
: ---
Assigned To: Panel Maintainers
Panel Maintainers
: 674714 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-06-05 20:59 UTC by Giovanni Campagna
Modified: 2012-07-15 22:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Clock Applet: port to modern GNOME technologies (143.42 KB, patch)
2012-06-05 21:00 UTC, Giovanni Campagna
none Details | Review
Clock Applet: port to modern GNOME technologies (148.57 KB, patch)
2012-06-08 11:34 UTC, Giovanni Campagna
committed Details | Review

Description Giovanni Campagna 2012-06-05 20:59:58 UTC
I had to do it because the broken dependency on libgweather is blocking 3.5.* from ever seeing light, and then I found it in a bad state of GConf, g_locale_from_utf8 back and forth, static buffers, manual signals instead of GBindings...
This should bring an improvement.
Comment 1 Giovanni Campagna 2012-06-05 21:00:27 UTC
Created attachment 215689 [details] [review]
Clock Applet: port to modern GNOME technologies

This is a big commit that updates the Clock Applet to use modern
libraries. This includes using GSettings instead of GConf, using
GDateTime instead of the equivalents from libc and using the new
API in libgweather.
Comment 2 Giovanni Campagna 2012-06-08 11:34:19 UTC
Created attachment 215933 [details] [review]
Clock Applet: port to modern GNOME technologies

This is a big commit that updates the Clock Applet to use modern
libraries. This includes using GSettings instead of GConf, using
GDateTime instead of the equivalents from libc and using the new
API in libgweather.

Tested now, seems to work.
Comment 3 Vincent Untz 2012-06-08 11:46:34 UTC
Hrm, I guess you don't have a series of patch instead? It'd been much easier to review each changes separately.
Comment 4 Giovanni Campagna 2012-06-08 11:50:35 UTC
No, it's one commit because it was just too messy to get individual changes in without breaking the build. Also, it is quite difficult to split for example the general GSettings port and the GSettings port for libgweather, or the general GSettings port or the GnomeWallClock rewrite (which in turn causes the other changes)
Comment 5 Vincent Untz 2012-06-21 12:05:50 UTC
Comment on attachment 215933 [details] [review]
Clock Applet: port to modern GNOME technologies

I really won't be able to properly review that for now, so I guess it's simpler to just commit and have testers before 3.6.0.
Comment 6 Giovanni Campagna 2012-06-21 17:31:26 UTC
It seems to work fine here, so I'm pushing it.

Attachment 215933 [details] pushed as cd1a0db - Clock Applet: port to modern GNOME technologies
Comment 7 André Klapper 2012-07-15 22:15:19 UTC
*** Bug 674714 has been marked as a duplicate of this bug. ***