On Tue, Mar 24, 2015 at 7:11 PM, Shawn Landden <sh...@churchofgit.com> wrote:
> /* Enforce the values of /etc/localtime */
> if (getenv("TZ")) {
> - fprintf(stderr, "Warning: Ignoring the TZ variable.\n\n");
> + fprintf(stderr, "Warning: Ignoring the %s variable.\n\n",
> "TZ");
What style is that? A second static string inserted with %s?
Wanto de-duplicate the first static string? :)
> - xstrftime(a, "%a %Y-%m-%d %H:%M:%S UTC", gmtime_r(&sec,
> &tm));
> + xstrftime(a, "%a %Y-%m-%d %H:%M:%S UTC", gmtime);
Why this change?
> + " RTC in UTC by calling 'timedatectl
> set-local-rtc 0'\n
> + " For more details see
> http://www.cl.cam.ac.uk/~mgk25/mswish/ut-rtc.html"; ANSI_HIGHLIGHT_OFF ".\n",
> stdout);
Hmm, I find links to "random" web pages in error output a bit too
unconventional.
Kay
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
