"Andrew Dunstan" <andrew@dunslane.net> writes:
> OTOH, I wonder if maybe what we previously did w.r.t. TZ didn't work, but
> the timezone libs were smart enough to disregard an empty TZ value.
The code in variable.c clearly does *not* work on machines where
putenv() copies what you hand it :-(. I suspect that no one noticed
because the clear_tz path isn't really used in normal situations.
I left it alone for the moment because I expect that we'll be ripping
it all out as soon as the dust settles with src/timezone. We can define
a less brain-dead API for testing validity of a timezone name ...
regards, tom lane