Bug 654970 - datetime: Add NTP support for SUSE variants
datetime: Add NTP support for SUSE variants
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2011-07-20 13:59 UTC by Vincent Untz
Modified: 2011-09-06 13:38 UTC (History)
1 user (show)

See Also:
GNOME target: ---
GNOME version: ---

Attachments
datetime: Add NTP support for SUSE variants (12.72 KB, patch)
2011-07-20 13:59 UTC, Vincent Untz 		committed Details | Diff | Review
View All Add an attachment (proposed patch, testcase, etc.)

Description Vincent Untz 2011-07-20 13:59:21 UTC 
This is nearly the same code as the Fedora one, except that the ntp service is called ntp (instead of ntpd), and the variable for the hardware clock is different.

Maybe we want to share some code there?
Comment 1 Vincent Untz 2011-07-20 13:59:23 UTC 
Created attachment 192307 [details] [review]
datetime: Add NTP support for SUSE variants
Comment 2 Vincent Untz 2011-07-20 14:04:52 UTC 
It's worth pointing out that we have some solid & generic code in a gdm patch in openSUSE to read/write /etc/sysconfig files. I don't know if it's of interest here (especially as we're slowly trying to kill those files with systemd).
Comment 3 Bastien Nocera 2011-09-06 13:38:14 UTC 
See bug 658352 about using systemd's D-Bus service directly.

Attachment 192307 [details] pushed as bf7c80d - datetime: Add NTP support for SUSE variants
Note You need to log in before you can comment on or make changes to this bug.