Skip to content

Commit

Permalink
Do not provide default NTP servers. Fixes systemd#437.
Browse files Browse the repository at this point in the history
Google's NTP servers do not provide UTC time but 'Google Time' so it is
not appropriate to use them as default. As we are not a vendor, do not
use a vendor pool. Instead, provide no default and require distributions
to provide ntp server values.
  • Loading branch information
aneeshusa committed Jul 1, 2015
1 parent 64ea3f9 commit c0adb05
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions configure.ac
Expand Up @@ -1028,11 +1028,12 @@ if test "x$enable_timesyncd" != "xno"; then
fi
AM_CONDITIONAL(ENABLE_TIMESYNCD, [test "$have_timesyncd" = "yes"])

AC_ARG_WITH(ntp-servers,
AS_HELP_STRING([--with-ntp-servers=NTPSERVERS],
[Space-separated list of default NTP servers]),
[NTP_SERVERS="$withval"],
[NTP_SERVERS="time1.google.com time2.google.com time3.google.com time4.google.com"])
AM_COND_IF([ENABLE_TIMESYNCD],
[AC_ARG_WITH([ntp-servers],
[AS_HELP_STRING([--with-ntp-servers=NTPSERVERS],
[Space-separated list of default NTP servers])],
[NTP_SERVERS="$withval"],
[AC_MSG_ERROR([Must provide ntp servers using --with-ntp-servers=NTPSERVERS for timesyncd])])])

AC_DEFINE_UNQUOTED(NTP_SERVERS, ["$NTP_SERVERS"], [Default NTP Servers])
AC_SUBST(NTP_SERVERS)
Expand Down

0 comments on commit c0adb05

Please sign in to comment.