From: "Michael Kerrisk (man-pages)" <mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: JWP <elseifthen-KK0ffGbhmjU@public.gmane.org>
Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH[V2] 3/5] tzset.3: correct system timezone file path
Date: Sun, 25 Jan 2015 15:11:22 +0100 [thread overview]
Message-ID: <54C4F98A.4070101@gmail.com> (raw)
In-Reply-To: <54BC11E2.9020906-KK0ffGbhmjU@public.gmane.org>
On 01/18/2015 09:04 PM, JWP wrote:
> The manual incorrectly states the system timezone
> path as /usr/share/zoneinfo/localtime. Glibc does
> not use that file for anything, it uses
> $(prefix)/etc/localtime.
>
> There is an ambiguous reference in the man-page to
> /etc/localtime, but it does not indicate that it
> will be used. It states clearly that
> /usr/share/zoneinfo/localtime is used.
>
> A comment in the glibc Makeconfig even says that
> file should not exist:
>
>> ... relative to $(zonedir). It is a good idea
>> to put this somewhere other than there, so the
>> zoneinfo directory contains only universal data,
>> localizing the configuration data elsewhere.
>
> Furthermore, the man-page does not indicate the
> reason this file is being used; which is because
> it is the configured system timezone.
>
> A later patch in this series address the
> possibility that /etc/localtime could be changed
> during glibc's compilation. The language changed
> in this patch points to the FILE section which is
> where the explanation will be. There is no reason
> to repeat that information multiple times
> throughout the man-page.
Thanks for the well-documented patch, William. Applied.
Cheers,
Michael
> EVIDENCE:
>
> glibc/Makeconfig:256: sysconfdir = $(prefix)/etc
> glibc/Makeconfig:272: localtime-file = $(sysconfdir)/localtime
>
> FROM INFO LIBC:
> C.2 Installing the C Library
> ============================
> To configure the locally used timezone, set the `TZ' environment
> variable. The script `tzselect' helps you to select the right value.
> As an example, for Germany, `tzselect' would tell you to use
> `TZ='Europe/Berlin''. For a system wide installation (the given paths
> are for an installation with `--prefix=/usr'), link the timezone file
> which is in `/usr/share/zoneinfo' to the file `/etc/localtime'. For
> Germany, you might execute `ln -s /usr/share/zoneinfo/Europe/Berlin
> /etc/localtime'.
>
> STEPS TO REPRODUCE:
>
> cd /usr/share/zoneinfo/
> cp Antarctica/South_Pole localtime
> cp /US/Eastern /etc/localtime
> cd
> date
> Sun Jan 18 12:06:36 EST 2015
> TZ=:/usr/share/zoneinfo/localtime date
> Mon Jan 19 06:06:54 NZDT 2015
> rm /etc/localtime
> date
> Sun Jan 18 17:08:37 UTC 2015
> TZ=:/usr/share/zoneinfo/localtime date
> Mon Jan 19 06:08:41 NZDT 2015
>
> Signed-off-by: J William Piggott <elseifthen-KK0ffGbhmjU@public.gmane.org>
> ---
>
> The following changes since commit b58839b96d626e67da233d4bf7ba04d5dfcd0e66:
>
> syscalls.2: Remove some details for sync_file_range2() (2015-01-18 19:56:05 +0100)
>
> are available in the git repository at:
>
> git-9UaJU3cA/F/QT0dZR+AlfA@public.gmane.org:jwpi/man-pages.git tzset
>
> for you to fetch changes up to 2b0725c0b6f6788628567ea4d5cfc396898584a0:
>
> tzset.3: TZ filespec omitts the colon (2015-01-18 14:40:21 -0500)
>
>
> man3/tzset.3 | 15 ++++++---------
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/man3/tzset.3 b/man3/tzset.3
> index 3f9b246..09fac15 100644
> --- a/man3/tzset.3
> +++ b/man3/tzset.3
> @@ -75,15 +75,12 @@ the year when daylight saving time applies).
> .PP
> If the
> .B TZ
> -variable does not appear in the environment, the \fItzname\fP
> -variable is initialized with the best approximation of local wall clock
> -time, as specified by the
> -.BR tzfile (5)-format
> -file \fIlocaltime\fP
> -found in the system timezone directory (see below).
> -(One also often sees
> -.I /etc/localtime
> -used here, a symlink to the right file in the system timezone directory.)
> +variable does not appear in the environment, the system timezone is used.
> +The system timezone is configured by copying, or linking, a file in the
> +.BR tzfile "(5) format to"
> +.IR /etc/localtime .
> +A timezone database of these files may be located in the system
> +timezone directory (see the \fBFILES\fP section below).
> .PP
> If the
> .B TZ
>
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-01-25 14:11 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-18 0:30 [PATCH][COVER LETTER] tzset.3: correct file paths and TZ use JWP
[not found] ` <54BAFE98.2090402-KK0ffGbhmjU@public.gmane.org>
2015-01-18 0:34 ` [PATCH] " JWP
[not found] ` <54BAFFAE.2020106-KK0ffGbhmjU@public.gmane.org>
2015-01-18 11:02 ` Michael Kerrisk (man-pages)
[not found] ` <54BB92AE.4090604-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-23 17:29 ` JWP
[not found] ` <54C28516.7070802-KK0ffGbhmjU@public.gmane.org>
2015-01-25 13:43 ` Michael Kerrisk (man-pages)
2015-01-18 20:00 ` [PATCH[V2] 1/5] tzset.3: filespec omitted incorrect JWP
[not found] ` <54BC10C7.1030706-KK0ffGbhmjU@public.gmane.org>
2015-01-25 14:11 ` Michael Kerrisk (man-pages)
2015-01-18 20:02 ` [PATCH[V2] 2/5] tzset.3: First TZ specification clarification JWP
[not found] ` <54BC1172.5060100-KK0ffGbhmjU@public.gmane.org>
2015-01-25 14:10 ` Michael Kerrisk (man-pages)
2015-01-18 20:04 ` [PATCH[V2] 3/5] tzset.3: correct system timezone file path JWP
[not found] ` <54BC11E2.9020906-KK0ffGbhmjU@public.gmane.org>
2015-01-25 14:11 ` Michael Kerrisk (man-pages) [this message]
2015-01-18 20:07 ` [PATCH[V2] 4/5] tzset.3: Add Environment section JWP
[not found] ` <54BC126E.1080904-KK0ffGbhmjU@public.gmane.org>
2015-01-25 14:12 ` Michael Kerrisk (man-pages)
[not found] ` <54C4F9E7.6090104-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-28 23:52 ` JWP
2015-01-29 0:09 ` [PATCH V3 1/1] " J William Piggott
[not found] ` <54C97A2D.2050908-KK0ffGbhmjU@public.gmane.org>
2015-01-29 5:50 ` Michael Kerrisk (man-pages)
[not found] ` <54C9CA11.207-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-29 16:27 ` [PATCH V4 " J William Piggott
[not found] ` <54CA5F5B.7070405-KK0ffGbhmjU@public.gmane.org>
2015-01-30 6:04 ` Michael Kerrisk (man-pages)
2015-01-29 0:13 ` [PATCH 1/1] tzset.3: Add description for posixrules file J William Piggott
[not found] ` <54C97B45.2070300-KK0ffGbhmjU@public.gmane.org>
2015-01-29 5:51 ` Michael Kerrisk (man-pages)
[not found] ` <54C9CA67.4080009-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-29 16:31 ` [PATCH V2 " J William Piggott
[not found] ` <54CA604A.1050500-KK0ffGbhmjU@public.gmane.org>
2015-01-30 6:04 ` Michael Kerrisk (man-pages)
2015-01-18 20:10 ` [PATCH[V2] 5/5] tzset.3: TZ filespec omitts the colon JWP
[not found] ` <54BC132A.8090504-KK0ffGbhmjU@public.gmane.org>
2015-01-25 14:11 ` Michael Kerrisk (man-pages)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54C4F98A.4070101@gmail.com \
--to=mtk.manpages-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=elseifthen-KK0ffGbhmjU@public.gmane.org \
--cc=linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.