Re: initdb failure in CVS

"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