pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
Cc: Andrew Gierth <andrew@tao11.riddles.org.uk>
Cc: Christoph Berg <myon@debian.org>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)
Date: Thu, 06 Jun 2019 12:51:30 -0400
Message-ID: <5519.1559839890@sss.pgh.pa.us> (raw)
In-Reply-To: <20190604164457.772invck3lgxk75s@alap3.anarazel.de>
References: <E1hK8qL-0005yH-VX@gemulon.postgresql.org>
	<20190604085735.GD24018@msg.df7cb.de>
	<87ftop8gtr.fsf@news-spur.riddles.org.uk>
	<20190604164457.772invck3lgxk75s@alap3.anarazel.de>

[ sorry for slow response, I'm on vacation ]

Andres Freund <andres@anarazel.de> writes:
> That makes sense. As far as I can tell the reason that 12 sometimes ends
> up with the proper timezone is that we shortcut the search by:

> 	/*
> 	 * Try to avoid the brute-force search by seeing if we can recognize the
> 	 * system's timezone setting directly.
> 	 *
> 	 * Currently we just check /etc/localtime; there are other conventions for
> 	 * this, but that seems to be the only one used on enough platforms to be
> 	 * worth troubling over.
> 	 */
> 	if (check_system_link_file("/etc/localtime", &tt, resultbuf))
> 		return resultbuf;

> which is actually a behaviour changing, rather than just an
> optimization, when there's a lot of equivalently scoring timezones.

Sure, that is intentionally a behavior change in this situation.
The theory is that if "Etc/UCT" is what the user put in /etc/localtime,
then that's the spelling she wants.  See 23bd3cec6.

But it seems to me that this code is *not* determining the result in
Christoph's case, because if it were, it'd be settling on Etc/UTC,
according to his followup report that

>> lrwxrwxrwx 1 root root 27 Mär 28 14:49 /etc/localtime -> /usr/share/zoneinfo/Etc/UTC

I'm not too familiar with what actually determines glibc's behavior
on Debian, but I'm suspicious that there's an inconsistency between
/etc/localtime and /etc/timezone.  We won't adopt the spelling we
see in /etc/localtime unless it agrees with the observed behavior of
localtime(3).

			regards, tom lane





view thread (74+ messages)  latest in thread

Message-ID: <5519.1559839890@sss.pgh.pa.us>
Permalink:  ../5519.1559839890@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/5519.1559839890@sss.pgh.pa.us

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, andres@anarazel.de, andrew@tao11.riddles.org.uk, myon@debian.org, pgsql-hackers@postgresql.org
  Subject: Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)
  In-Reply-To: <5519.1559839890@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox