Hello,
even if 7.4 is almost at the end of it's lifetime i found a bug/problem
on a fairly new laptop (not surprising: during upgrade tests). All
timestamp regression tests failed, only "th" was emitted.
I tracked the problem done to the function str_numth() in
src/backend/utils/adt/formatting.c. The fix (attached) is easy: i
stole the function code from the latest 8.0 version which looks fare
more sane in handling the string pointers.
Since this patch is easy enough (replacing one line with three
lines of code), does not seem to break existing code and removes at
least one bug i ask to include this patch into the next 7.4 release (i
assume there will be at least one release if 7.4 reaches eol).
Thanks & kind regards
--
Andreas 'ads' Scherbaum
German PostgreSQL User Group
European PostgreSQL User Group - Board of Directors
*** src/bin/psql/describe.c 2008-12-21 13:19:40.000000000 +0100
--- src/bin/psql/describe.c.orig 2008-12-21 02:16:31.000000000 +0100
***************
*** 464,476 ****
" d.datctype as \"%s\",\n",
gettext_noop("Collation"),
gettext_noop("Ctype"));
! if (pset.sversion >= 80100)
! appendPQExpBuffer(&buf,
! " pg_catalog.array_to_string(d.datacl, E'\\n') as \"%s\"",
! gettext_noop("Access Privileges"));
! else
! appendPQExpBuffer(&buf,
! " pg_catalog.array_to_string(d.datacl, '\\n') as \"%s\"",
gettext_noop("Access Privileges"));
if (verbose && pset.sversion >= 80200)
appendPQExpBuffer(&buf,
--- 464,471 ----
" d.datctype as \"%s\",\n",
gettext_noop("Collation"),
gettext_noop("Ctype"));
! appendPQExpBuffer(&buf,
! " d.datacl as \"%s\"",
gettext_noop("Access Privileges"));
if (verbose && pset.sversion >= 80200)
appendPQExpBuffer(&buf,
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
